Merge pull request #213 from Zyzyx/main

some light comment clean up
release-engineering · Oct 2, 2024 · 3cd20c0 · 3cd20c0
2 parents f6c88df + c4fa8a5
commit 3cd20c0
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 9 deletions.
diff --git a/sync2jira/upstream_issue.py b/sync2jira/upstream_issue.py
@@ -64,8 +64,8 @@ def handle_github_message(msg, config, pr_filter=True):
     owner = msg['msg']['repository']['owner']['login']
     repo = msg['msg']['repository']['name']
     upstream = '{owner}/{repo}'.format(owner=owner, repo=repo)
-    mapped_repos = config['sync2jira']['map']['github']
 
+    mapped_repos = config['sync2jira']['map']['github']
     if upstream not in mapped_repos:
         log.debug("%r not in Github map: %r", upstream, mapped_repos.keys())
         return None

diff --git a/sync2jira/upstream_pr.py b/sync2jira/upstream_pr.py
@@ -46,12 +46,10 @@ def handle_github_message(msg, config, suffix):
     :returns: Issue object
     :rtype: sync2jira.intermediary.PR
     """
-    # Create our title (i.e. owner/repo)
     owner = msg['msg']['repository']['owner']['login']
     repo = msg['msg']['repository']['name']
     upstream = '{owner}/{repo}'.format(owner=owner, repo=repo)
 
-    # Check if upstream is in mapped repos
     mapped_repos = config['sync2jira']['map']['github']
     if upstream not in mapped_repos:
         log.debug("%r not in Github map: %r", upstream, mapped_repos.keys())

diff --git a/tests/test_upstream_issue.py b/tests/test_upstream_issue.py
@@ -81,15 +81,15 @@ def setUp(self):
             }
         }
 
-        # Mock Github Reporter
+        # Mock GitHub Reporter
         self.mock_github_person = MagicMock()
         self.mock_github_person.name = 'mock_name'
 
-        # Mock Github Repo
+        # Mock GitHub Repo
         self.mock_github_repo = MagicMock()
         self.mock_github_repo.get_issue.return_value = self.mock_github_issue
 
-        # Mock Github Client
+        # Mock GitHub Client
         self.mock_github_client = MagicMock()
         self.mock_github_client.get_repo.return_value = self.mock_github_repo
         self.mock_github_client.get_user.return_value = self.mock_github_person

diff --git a/tests/test_upstream_pr.py b/tests/test_upstream_pr.py
@@ -81,16 +81,16 @@ def setUp(self):
             }
         }
 
-        # Mock Github Reporter
+        # Mock GitHub Reporter
         self.mock_github_person = MagicMock()
         self.mock_github_person.name = 'mock_name'
 
-        # Mock Github Repo
+        # Mock GitHub Repo
         self.mock_github_repo = MagicMock()
         self.mock_github_repo.get_pull.return_value = self.mock_github_pr
         self.mock_github_repo.get_issue.return_value = self.mock_github_pr
 
-        # Mock Github Client
+        # Mock GitHub Client
         self.mock_github_client = MagicMock()
         self.mock_github_client.get_repo.return_value = self.mock_github_repo
         self.mock_github_client.get_user.return_value = self.mock_github_person