Remove all print statements

Author: jackzhxng

.github/scripts/check_labels.py

@@ -12,10 +12,7 @@
 
 def delete_all_label_err_comments(pr: "GitHubPR") -> None:
     for comment in pr.get_comments():
-        print(f"comment: {comment}")
         if is_label_err_comment(comment):
-            print("is label err comment")
-            print(f"comment.database_id: {comment.database_id}")
             gh_delete_comment(pr.org, pr.project, comment.database_id)
 
 
@@ -41,14 +38,10 @@ def parse_args() -> Any:
 
 
 def main() -> None:
-    print("main")
     args = parse_args()
     repo = GitRepo(get_git_repo_dir(), get_git_remote_name())
     org, project = repo.gh_owner_and_name()
     pr = GitHubPR(org, project, args.pr_num)
-    print(f"repo: {repo}")
-    print(f"org, project: {org}, {project}")
-    print(f"pr: {pr}")
 
     try:
         if not has_required_labels(pr):
@@ -57,15 +50,11 @@ def main() -> None:
             if args.exit_non_zero:
                 raise RuntimeError("PR does not have required labels")
         else:
-            print("pass")
             delete_all_label_err_comments(pr)
     except Exception as e:
-        print("general exception")
-        print(e)
         if args.exit_non_zero:
             raise RuntimeError(f"Error checking labels: {e}") from e
 
-    print("success")
     sys.exit(0)
 
 

.github/scripts/github_utils.py

@@ -66,19 +66,6 @@ def gh_fetch_url_and_headers(
         raise
 
 
-# def gh_fetch_url(
-#     url: str,
-#     *,
-#     headers: Optional[Dict[str, str]] = None,
-#     data: Union[Optional[Dict[str, Any]], str] = None,
-#     method: Optional[str] = None,
-#     reader: Callable[[Any], Any] = lambda x: x.read(),
-# ) -> Any:
-#     print(f"api return conent: {gh_fetch_url_and_headers(url, headers=headers, data=data, reader=json.load, method=method)}")
-#     return gh_fetch_url_and_headers(
-#         url, headers=headers, data=data, reader=json.load, method=method
-#     )[1]
-
 def gh_fetch_url(
     url: str,
     *,
@@ -180,12 +167,6 @@ def gh_post_commit_comment(
     )
 
 
-# def gh_delete_comment(org: str, repo: str, comment_id: int) -> None:
-#     print("deleting comment")
-#     url = f"{GITHUB_API_URL}/repos/{org}/{repo}/issues/comments/{comment_id}"
-#     print(f"url: {url}")
-#     gh_fetch_url(url, method="DELETE")
-
 def gh_delete_comment(org: str, repo: str, comment_id: int) -> None:
     url = f"{GITHUB_API_URL}/repos/{org}/{repo}/issues/comments/{comment_id}"
     gh_fetch_url(url, method="DELETE", reader=lambda x: x.read())

.github/scripts/label_utils.py

@@ -64,7 +64,6 @@ def get_last_page_num_from_header(header: Any) -> int:
 @lru_cache
 def gh_get_labels(org: str, repo: str) -> List[str]:
     prefix = f"https://api.github.com/repos/{org}/{repo}/labels?per_page=100"
-    print(f"prefix: {prefix}")
     header, info = request_for_labels(prefix + "&page=1")
     labels: List[str] = []
     update_labels(labels, info)
@@ -77,8 +76,6 @@ def gh_get_labels(org: str, repo: str) -> List[str]:
         _, info = request_for_labels(prefix + f"&page={page_number}")
         update_labels(labels, info)
 
-    print(f"all labels on gh: {labels}")
-
     return labels
 
 
@@ -115,15 +112,11 @@ def get_release_notes_labels(org: str, repo: str) -> List[str]:
 
 
 def has_required_labels(pr: "GitHubPR") -> bool:
-    print(f"pr: {pr}, pr.org: {pr.org}, pr.project: {pr.project}")
     pr_labels = pr.get_labels()
-    print(f"pr_labels: {pr_labels}")
     # Check if PR is not user facing
     is_not_user_facing_pr = any(
         label.strip() == "topic: not user facing" for label in pr_labels
     )
-    print(f"is_not_user_facing_pr: {is_not_user_facing_pr}")
-    print(f"release note labels: {get_release_notes_labels(pr.org, pr.project)}")
     return is_not_user_facing_pr or any(
         label.strip() in get_release_notes_labels(pr.org, pr.project)
         for label in pr_labels

.github/workflows/check-labels.yml

@@ -16,7 +16,7 @@ on:
   # a PR that targets a gh/**/base branch.
   pull_request:
     types: [opened, synchronize, reopened, labeled, unlabeled]
-    branches: [gh/**/base, main]
+    branches: [gh/**/base]
 
   workflow_dispatch:
     inputs: