Skip to content

chore: update assign-reviewers workflow to be hermetic - #9037

Open
shivanee-p wants to merge 8 commits into
mainfrom
shivaneep-assign-reviewers
Open

chore: update assign-reviewers workflow to be hermetic#9037
shivanee-p wants to merge 8 commits into
mainfrom
shivaneep-assign-reviewers

Conversation

@shivanee-p

@shivanee-p shivanee-p commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Updated reviewer workflow to be almost fully hermetic to prevent failures from external API calls

  • Added actions/checkout (fetch-depth: 0) to execute workflow tasks locally.
  • Replaced GitHub REST API calls with local file parsing and local git diff --name-only.
  • Replaced open PR workload API queries PR-number modulus selection reviewer distribution without network API dependency.
  • Updated team_members: comment definition in .github/CODEOWNERS
  • The only external api calls are now to assign/remove reviewers

@shivanee-p
shivanee-p requested a review from a team as a code owner July 31, 2026 17:18
@github-actions
github-actions Bot requested a review from westarle July 31, 2026 17:19

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the .github/CODEOWNERS file by replacing the team member username @pearigee with @westarle in a comment. There are no review comments, and I have no feedback to provide.

@shivanee-p
shivanee-p marked this pull request as draft July 31, 2026 17:25
@shivanee-p
shivanee-p removed request for a team and westarle July 31, 2026 17:25
@shivanee-p
shivanee-p marked this pull request as ready for review July 31, 2026 19:15
@shivanee-p
shivanee-p force-pushed the shivaneep-assign-reviewers branch from 16dc66f to 6168bf5 Compare July 31, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant