Skip to content

Enable automation of PR review assignments across repos #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
clyne opened this issue Jun 3, 2021 · 2 comments · Fixed by #105
Closed

Enable automation of PR review assignments across repos #79

clyne opened this issue Jun 3, 2021 · 2 comments · Fixed by #105
Assignees
Labels
infrastructure Infrastructure related issue

Comments

@clyne
Copy link
Contributor

clyne commented Jun 3, 2021

Investigate and hopefully enable GitHub's "round robin" reviewer assignment for all repos as described here:

https://docs.github.com/en/organizations/organizing-members-into-teams/managing-code-review-assignment-for-your-team#routing-algorithms

@clyne clyne added the infrastructure Infrastructure related issue label Jun 3, 2021
@clyne
Copy link
Contributor Author

clyne commented Jun 3, 2021

Need guidance from @kmpaul on how this working. @andersy005 believes the RR algo is tied to the teams: @ProjectPythia/education and @ProjectPythia/infrastructure. I.e. if one of these teams is tagged, automated RR scheduling will be used. The process is NOT completely automated. The PR submitter has to select one of the teams before scheduling kicks off. Could we further automate? I.e. if tagged infrastructure the "infrastructure" team is selected.

N.B. configuration is here: https://github.com/orgs/ProjectPythia/teams/infrastructure/edit/review_assignment

and https://github.com/orgs/ProjectPythia/teams/education/edit/review_assignment

@clyne clyne added this to the InitialPortalRelease milestone Jun 4, 2021
@dopplershift
Copy link
Contributor

I think we can get this completely automated by adding a team to CODEOWNERS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Infrastructure related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants