Very interesting one - as an outsider I am interested to see how this initiative will work out for the beam community.
Just one comment - maybe you do not know but in GitHub there is a "CODEOWNERS" feature (I notice you are not using it). Quote from https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners | Code owners are automatically requested for review when someone opens a pull request that modifies code that they own. Code owners are not automatically requested to review draft pull requests. For more information about draft pull requests, see "About pull requests." When you mark a draft pull request as ready for review, code owners are automatically notified. If you convert a pull request to a draft, people who are already subscribed to notifications are not automatically unsubscribed. For more information, see "Changing the stage of a pull request." This is an extremely poor version of what you try to do in Beam (just assign everyone who is code owner as reviewer, no round-robin, no reviewers role etc.), but maybe you want to try it quickly if you want to test if any kind of "ownership" might help with at least initial vetting of PRs. This feature is enabled by literally committing one - gitignore-like - file to repo, so it can be introduced extremely quickly. Airlfow's CODEOWNERS here as an example: https://github.com/apache/airflow/blob/main/.github/CODEOWNERS J. On Thu, Feb 10, 2022 at 7:31 AM Ahmet Altay <al...@google.com> wrote: > Thank you Danny. I think this is a great problem to solve, and the > proposal looks great too :) I added comments as others but overall I like > it. > > On Wed, Feb 9, 2022 at 3:02 PM Brian Hulette <bhule...@google.com> wrote: > >> Thanks Danny! I left a few suggestions in the doc but I very much like >> this idea overall. >> >> I especially like that "reviewers" is orthogonal to "committers", giving >> new contributors a clear way to volunteer to help out with code reviews. If >> we do this we should document it in the contribution guide [1]. >> >> [1] https://beam.apache.org/contribute/ >> >> On Wed, Feb 9, 2022 at 2:54 PM Kerry Donny-Clark <kerr...@google.com> >> wrote: >> >>> Danny, this looks like a great mechanism to ensure we review PRs quickly >>> and distribute the review work more evenly. >>> Thanks for outlining a clear plan. I strongly support this. >>> Kerry >>> >>> On Wed, Feb 9, 2022, 5:16 PM Danny McCormick <dannymccorm...@google.com> >>> wrote: >>> >>>> Hey everyone, I put together a design doc for automating the assignment >>>> of reviewers in Beam pull requests. I'd appreciate any thoughts you have! >>>> >>>> Right now, we don't have a well defined automated system for staying on >>>> top of pull request reviews - we rely on contributors being able to find >>>> the correct OWNERS file and committers manually triaging/calling attention >>>> to old pull requests. This doc proposes adding automation driven by GitHub >>>> Actions to automatically round robin new PR reviews to a set of >>>> contributors, thus balancing the load. It also proposes adding a new role >>>> within the beam community of a reviewer who is responsible for an >>>> initial code review on some PRs before they are routed to a committer for >>>> final review. >>>> >>>> Please share any feedback or support here - >>>> https://docs.google.com/document/d/1FhRPRD6VXkYlLAPhNfZB7y2Yese2FCWBzjx67d3TjBo/edit?usp=sharing >>>> >>>> Thanks, >>>> Danny >>>> >>>
