Skip to content

fix(scripts): offer repo teams as reviewers in make pr - #368

Merged
gmmcosta15 merged 3 commits into
devfrom
fix/create-pr-team-reviewers
Oct 23, 2026
Merged

gmmcosta15 merged 3 commits into
devfrom
fix/create-pr-team-reviewers

Conversation

@gmmcosta15

@gmmcosta15 gmmcosta15 commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Description

  • Feature
  • Bug Fix
  • Code Refactor
  • Documentation

make pr builds its reviewer list from repos/{repo}/collaborators, an endpoint that only returns users, so BlocksTechnology/devteam (the team every PR requests) could never be picked. It had to be added on GitHub after the PR was open. This PR adds the repo's teams to the top of the reviewer list as owner/slug, which gh pr edit --add-reviewer accepts unchanged.

Changes

  • create-pr.sh: list repos/{repo}/teams before the collaborators in the Reviewers picker; skipped silently when the call fails (personal repos have no teams)

Motivation

A team request reaches every member, so picking it is the common case.

Tests

  • shellcheck scripts/create-pr.sh clean; bash -n ok
  • The patched block run against BlocksTechnology/BlocksScreen lists BlocksTechnology/devteam; against a repo that returns 404 it adds nothing and prints nothing

@gmmcosta15 gmmcosta15 added bug Something is broken, or a fix for it ci CI/CD workflows, release and PR tooling labels Oct 23, 2026
@gmmcosta15 gmmcosta15 self-assigned this Oct 23, 2026
@gmmcosta15
gmmcosta15 requested a review from a team October 23, 2026 09:08
@gmmcosta15
gmmcosta15 merged commit b8d1171 into dev Oct 23, 2026
9 checks passed
@gmmcosta15
gmmcosta15 deleted the fix/create-pr-team-reviewers branch October 23, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is broken, or a fix for it ci CI/CD workflows, release and PR tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants