From a04a65e66b46b1bcdf1da914b4cec18b7f4797af Mon Sep 17 00:00:00 2001 From: rnetser Date: Tue, 23 Jul 2024 16:12:59 +0300 Subject: [PATCH] wip label - add missing f-string --- webhook_server_container/libs/github_api.py | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/webhook_server_container/libs/github_api.py b/webhook_server_container/libs/github_api.py index 392e619a..f69c9132 100644 --- a/webhook_server_container/libs/github_api.py +++ b/webhook_server_container/libs/github_api.py @@ -1706,7 +1706,10 @@ def process_opened_or_synchronize_pull_request(self) -> None: with ThreadPoolExecutor() as executor: prepare_pull_futures.append(executor.submit(self.assign_reviewers)) prepare_pull_futures.append( - executor.submit(self._add_label, **{"label": f"{BRANCH_LABEL_PREFIX}{self.pull_request_branch}"}) + executor.submit( + self._add_label, + **{"label": f"{BRANCH_LABEL_PREFIX}{self.pull_request_branch}"}, + ) ) prepare_pull_futures.append(executor.submit(self.label_pull_request_by_merge_state)) prepare_pull_futures.append(executor.submit(self.set_merge_check_queued)) @@ -1909,8 +1912,14 @@ def get_all_required_status_checks(self) -> List[str]: return _all_required_status_checks def set_wip_label_based_on_title(self) -> None: - if self.pull_request.title.lower().startswith("{WIP_STR}:"): + if self.pull_request.title.lower().startswith(f"{WIP_STR}:"): + self.logger.debug( + f"{self.log_prefix} Found {WIP_STR} in {self.pull_request.title}; adding {WIP_STR} label." + ) self._add_label(label=WIP_STR) else: + self.logger.debug( + f"{self.log_prefix} {WIP_STR} not found in {self.pull_request.title}; removing {WIP_STR} label." + ) self._remove_label(label=WIP_STR)