refactor: get_issue_url() returns *just* the URL

Leave the extra text to the caller (main)
This commit is contained in:
Robert Alonso 2024-10-26 20:22:18 +00:00
parent b9c9d197ff
commit 1674ad54d4
2 changed files with 2 additions and 2 deletions

View File

@ -352,4 +352,4 @@ class GitHubClient(object):
return pr_request.status_code return pr_request.status_code
def get_issue_url(self, new_issue_number): 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}'

View File

@ -101,7 +101,7 @@ if __name__ == "__main__":
# Duplicate the line to retain the comment syntax. # Duplicate the line to retain the comment syntax.
new_line = file_lines[line_number] new_line = file_lines[line_number]
remove = fr'{raw_issue.identifier}.*{raw_issue.title}' 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) new_line = re.sub(remove, insert, new_line)
# Check if the URL line already exists, if so abort. # 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: if line_number == len(file_lines) - 1 or file_lines[line_number + 1] != new_line: