From 59ad8aeda6352e5da12aaab08ede719ac3a832e5 Mon Sep 17 00:00:00 2001 From: Michael Kubacki Date: Wed, 31 Jul 2024 19:06:24 -0400 Subject: .github/request-reviews.yml: Formatting (non-functional) Updates code for PEP8 formatting by using the Black code formatter. Signed-off-by: Michael Kubacki --- .github/scripts/GitHub.py | 12 ++++++++---- .github/workflows/request-reviews.yml | 24 ++++++++++++++++++------ 2 files changed, 26 insertions(+), 10 deletions(-) diff --git a/.github/scripts/GitHub.py b/.github/scripts/GitHub.py index ed39474064..628cd8412a 100644 --- a/.github/scripts/GitHub.py +++ b/.github/scripts/GitHub.py @@ -126,7 +126,9 @@ def get_reviewers_for_range( ) if cmd_ret != 0: print( - f"::error title=Reviewer Lookup Error!::Error calling GetMaintainer.py: [{cmd_ret}]: {reviewer_stream_buffer.getvalue()}" + f"::error title=Reviewer Lookup Error!::Error calling " + f"GetMaintainer.py: [{cmd_ret}]: " + f"{reviewer_stream_buffer.getvalue()}" ) return [] @@ -138,7 +140,8 @@ def get_reviewers_for_range( return [] print( - f"::debug title=Commit {commit_sha[:7]} Reviewer(s)::{', '.join(matches)}" + f"::debug title=Commit {commit_sha[:7]} " + f"Reviewer(s)::{', '.join(matches)}" ) raw_reviewers.extend(matches) @@ -232,8 +235,9 @@ def add_reviewers_to_pr( # Notify the admins of the repository if non-collaborators are requested. if non_collaborators: print( - f"::warning title=Non-Collaborator Reviewers Found!::{', '.join(non_collaborators)}" - ) + f"::warning title=Non-Collaborator Reviewers Found!::" + f"{', '.join(non_collaborators)}" + ) for comment in pr.get_issue_comments(): # If a comment has already been made for these non-collaborators, diff --git a/.github/workflows/request-reviews.yml b/.github/workflows/request-reviews.yml index bd00dd4516..15de2cb0e5 100644 --- a/.github/workflows/request-reviews.yml +++ b/.github/workflows/request-reviews.yml @@ -68,24 +68,36 @@ jobs: import git import os import sys - sys.path.append(os.path.join(os.environ['WORKSPACE_PATH'], ".github")) + + sys.path.append(os.path.join(os.environ["WORKSPACE_PATH"], ".github")) from scripts import GitHub - WORKSPACE_PATH = os.environ['WORKSPACE_PATH'] - GET_MAINTAINER_LOCAL_PATH = os.path.join(WORKSPACE_PATH, os.environ['GET_MAINTAINER_REL_PATH']) + WORKSPACE_PATH = os.environ["WORKSPACE_PATH"] + GET_MAINTAINER_LOCAL_PATH = os.path.join( + WORKSPACE_PATH, os.environ["GET_MAINTAINER_REL_PATH"] + ) # Step 1: Get the GitHub created PR commit SHA (contains all changes in a single commit) - pr_commit_sha = GitHub.get_pr_sha(os.environ['GH_TOKEN'], os.environ['ORG_NAME'], os.environ['REPO_NAME'], int(os.environ['PR_NUMBER'])) + pr_commit_sha = GitHub.get_pr_sha( + os.environ["GH_TOKEN"], + os.environ["ORG_NAME"], + os.environ["REPO_NAME"], + int(os.environ["PR_NUMBER"]), + ) if not pr_commit_sha: sys.exit(1) - print(f"::notice title=PR Commit SHA::Looking at files in consolidated PR commit: {pr_commit_sha}") + print( + f"::notice title=PR Commit SHA::Looking at files in consolidated PR commit: {pr_commit_sha}" + ) # Step 2: Fetch only the PR commit to get the files changed in the PR git.Repo(WORKSPACE_PATH).remotes.origin.fetch(pr_commit_sha, depth=1) # Step 3: Get the list of reviewers for the PR - reviewers = GitHub.get_reviewers_for_range(WORKSPACE_PATH, GET_MAINTAINER_LOCAL_PATH, pr_commit_sha, pr_commit_sha) + reviewers = GitHub.get_reviewers_for_range( + WORKSPACE_PATH, GET_MAINTAINER_LOCAL_PATH, pr_commit_sha, pr_commit_sha + ) if not reviewers: print("::notice title=No New Reviewers Found!::No reviewers found for this PR.") sys.exit(0) -- cgit