Skip to content

Commit

Permalink
Consider both commit authors and PR authors in status of testing issue (
Browse files Browse the repository at this point in the history
apache#43192)

* Consider both commit authors and PR authors in status of testing issue

* Consider both commit authors and PR authors in status of testing issue
  • Loading branch information
amoghrajesh authored Oct 20, 2024
1 parent 930c0d7 commit 04368b7
Showing 1 changed file with 16 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3476,6 +3476,22 @@ def generate_issue_content(
continue

pull_requests[pr_number] = pr

# retrieve and append commit authors (to handle cherry picks)
if hasattr(pr, "get_commits"):
try:
commits = pr.get_commits()
for commit in commits:
author = commit.author
if author:
users[pr_number].add(author.login)
progress.console.print(f"Added commit author {author.login} for PR#{pr_number}")

except Exception as e:
progress.console.print(
f"[warn]Could not retrieve commits for PR#{pr_number}: {e}, skipping[/]"
)

# GitHub does not have linked issues in PR - but we quite rigorously add Fixes/Closes
# Relate so we can find those from the body
if pr.body:
Expand Down

0 comments on commit 04368b7

Please sign in to comment.