From 1674ad54d4d1566b6ca18cd2fd2b643e0211dd05 Mon Sep 17 00:00:00 2001 From: Robert Alonso <17463757+rgalonso@users.noreply.github.com> Date: Sat, 26 Oct 2024 20:22:18 +0000 Subject: [PATCH] refactor: get_issue_url() returns *just* the URL Leave the extra text to the caller (main) --- GitHubClient.py | 2 +- main.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/GitHubClient.py b/GitHubClient.py index dea1daa..f38db91 100644 --- a/GitHubClient.py +++ b/GitHubClient.py @@ -352,4 +352,4 @@ class GitHubClient(object): return pr_request.status_code def get_issue_url(self, new_issue_number): - return f'Issue URL: {self.line_base_url}{self.repo}/issues/{new_issue_number}' + return f'{self.line_base_url}{self.repo}/issues/{new_issue_number}' diff --git a/main.py b/main.py index b0dd63f..a16f753 100644 --- a/main.py +++ b/main.py @@ -101,7 +101,7 @@ if __name__ == "__main__": # Duplicate the line to retain the comment syntax. new_line = file_lines[line_number] remove = fr'{raw_issue.identifier}.*{raw_issue.title}' - insert = client.get_issue_url(new_issue_number) + insert = f'Issue URL: {client.get_issue_url(new_issue_number)}' new_line = re.sub(remove, insert, new_line) # Check if the URL line already exists, if so abort. if line_number == len(file_lines) - 1 or file_lines[line_number + 1] != new_line: