Merge pull request #246 from creativecommons/error-handling-and-cleanup

Improve error handling and minor clean-up

Author: TimidRobot

README.md

@@ -114,7 +114,6 @@ in a GitHub project:
 [manage_issues]: .github/workflows/manage_issues.yml
 [manage_new_issues]: manage_new_issues_and_pull_requests.py
 [proj_possumbilities]: https://github.com/orgs/creativecommons/projects/23/views/1
-[proj-shafiya-heena]: https://github.com/orgs/creativecommons/projects/22/views/1
 [proj_timidrobot]: https://github.com/orgs/creativecommons/projects/15/views/1
 
 

_ARCHIVE/move_issues.py

@@ -185,9 +185,7 @@ def get_shafiya_items(github_gql_client):
             labels = []
             for label_edge in node["labels"]["edges"]:
                 labels.append(label_edge["node"]["name"])
-            items["issues"].append(
-                [repo, number, created, item_id]
-            )
+            items["issues"].append([repo, number, created, item_id])
         elif type_ == "PullRequest":
             items["prs"].append([repo, number, created, item_id])
     items["issues"].sort()

ccos/teams/set_codeowners.py

@@ -7,6 +7,7 @@
 
 # Third-party
 import git
+from github.GithubException import UnknownObjectException
 
 # First-party/Local
 from ccos.gh_utils import (
@@ -108,7 +109,11 @@ def check_and_fix_repo(args, organization, repo_name, teams, temp_dir):
 
     LOG.info(f"Checking and fixing {repo_name}...")
     repo_dir = os.path.join(temp_dir, repo_name)
-    gh_repo = organization.get_repo(repo_name)
+    try:
+        gh_repo = organization.get_repo(repo_name)
+    except UnknownObjectException:
+        LOG.error(f"Repository not found: {repo_name}")
+        raise
     clone_url = get_github_repo_url_with_credentials(repo_name)
     local_repo = set_up_repo(clone_url, repo_dir)
     codeowners_path = Path(os.path.join(repo_dir, ".github", "CODEOWNERS"))

sync_community_teams.py

@@ -38,8 +38,7 @@ def setup():
         "-d",
         "--debug",
         action="store_true",
-        help="Debug mode: show differences instead of pushing commits and"
-        " creating pull requests",
+        help="Debug mode: show differences instead of making changes",
     )
     args = ap.parse_args()
     return args