diff options
author | Michael Kubacki <michael.kubacki@microsoft.com> | 2024-07-31 19:06:24 -0400 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2024-08-05 19:30:26 +0000 |
commit | 59ad8aeda6352e5da12aaab08ede719ac3a832e5 (patch) | |
tree | 017139fcc50f77b97dbe187da2dd29619e89638c | |
parent | 32a099c358b3d4b093f76b2d060bcb3154ac5c56 (diff) | |
download | edk2-59ad8aeda6352e5da12aaab08ede719ac3a832e5.tar.gz edk2-59ad8aeda6352e5da12aaab08ede719ac3a832e5.tar.bz2 edk2-59ad8aeda6352e5da12aaab08ede719ac3a832e5.zip |
.github/request-reviews.yml: Formatting (non-functional)
Updates code for PEP8 formatting by using the Black code formatter.
Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
-rw-r--r-- | .github/scripts/GitHub.py | 12 | ||||
-rw-r--r-- | .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)
|