While I like the idea of having a CODEOWNERS file, the Github implementation is lacking: 1. Reviewers are automatically assigned at each push. 2. Reviewer assignment can be excessive (e.g. 5 reviewers in Eugene's PR 5940). 3. Non-committers aren't assigned as reviewers. 4. Non-committers can't change the list of reviewers.
I propose renaming the file to disable the auto-reviewer assignment feature. In its place I'll add a script that suggests reviewers. On Fri, Jul 13, 2018 at 9:09 AM Udi Meiri <eh...@google.com> wrote: > Hi Etienne, > > Yes you could be as precise as you want. The paths I listed are just > suggestions. :) > > > On Fri, Jul 13, 2018 at 1:12 AM Jean-Baptiste Onofré <j...@nanthrax.net> > wrote: > >> Hi, >> >> I think it's already do-able just providing the expected path. >> >> It's a good idea especially for the core. >> >> Regards >> JB >> >> On 13/07/2018 09:51, Etienne Chauchot wrote: >> > Hi Udi, >> > >> > I also have a question, related to what Eugene asked : I see that the >> > code paths are the ones of the modules. Can we be more precise than that >> > to assign reviewers ? As an example, I added myself to runner/core >> > because I wanted to take a look at the PRs related to >> > runner/core/metrics but I'm getting assigned to all runner-core PRs. Can >> > we specify paths like >> > runners/core-java/src/main/java/org/apache/beam/runners/core/metrics ? >> > I know it is a bit too precise so a bit risky, but in that particular >> > case, I doubt that the path will change. >> > >> > Etienne >> > >> > Le jeudi 12 juillet 2018 à 16:49 -0700, Eugene Kirpichov a écrit : >> >> Hi Udi, >> >> >> >> I see that the PR was merged - thanks! However it seems to have some >> >> unintended effects. >> >> >> >> On my PR https://github.com/apache/beam/pull/5940 , I assigned a >> >> reviewer manually, but the moment I pushed a new commit, it >> >> auto-assigned a lot of other people to it, and I had to remove them. >> >> This seems like a big inconvenience to me, is there a way to disable >> this? >> >> >> >> Thanks. >> >> >> >> On Thu, Jul 12, 2018 at 2:53 PM Udi Meiri <eh...@google.com >> >> <mailto:eh...@google.com>> wrote: >> >>> :/ That makes it a little less useful. >> >>> >> >>> On Thu, Jul 12, 2018 at 11:14 AM Tim Robertson >> >>> <timrobertson...@gmail.com <mailto:timrobertson...@gmail.com>> wrote: >> >>>> Hi Udi >> >>>> >> >>>> I asked the GH helpdesk and they confirmed that only people with >> >>>> write access will actually be automatically chosen. >> >>>> >> >>>> It don't expect it should stop us using it, but we should be aware >> >>>> that there are non-committers also willing to review. >> >>>> >> >>>> Thanks, >> >>>> Tim >> >>>> >> >>>> On Thu, Jul 12, 2018 at 7:24 PM, Mikhail Gryzykhin >> >>>> <mig...@google.com <mailto:mig...@google.com>> wrote: >> >>>>> Idea looks good in general. >> >>>>> >> >>>>> Did you look into ways to keep this file up-to-date? For example we >> >>>>> can run monthly job to see if owner was active during this period. >> >>>>> >> >>>>> --Mikhail >> >>>>> >> >>>>> Have feedback <http://go/migryz-feedback>? >> >>>>> >> >>>>> >> >>>>> On Thu, Jul 12, 2018 at 9:56 AM Udi Meiri <eh...@google.com >> >>>>> <mailto:eh...@google.com>> wrote: >> >>>>>> Thanks all! >> >>>>>> I'll try to get the file merged today and see how it works out. >> >>>>>> Please surface any issues, such as with auto-assignment, here or >> >>>>>> in JIRA. >> >>>>>> >> >>>>>> On Thu, Jul 12, 2018 at 2:12 AM Etienne Chauchot >> >>>>>> <echauc...@apache.org <mailto:echauc...@apache.org>> wrote: >> >>>>>>> Hi, >> >>>>>>> >> >>>>>>> I added myself as a reviewer for some modules. >> >>>>>>> >> >>>>>>> Etienne >> >>>>>>> >> >>>>>>> Le lundi 09 juillet 2018 à 17:06 -0700, Udi Meiri a écrit : >> >>>>>>>> Hi everyone, >> >>>>>>>> >> >>>>>>>> I'm proposing to add auto-reviewer-assignment using Github's >> >>>>>>>> CODEOWNERS mechanism. >> >>>>>>>> Initial version is >> >>>>>>>> here: _https://github.com/apache/beam/pull/5909/files_ >> >>>>>>>> >> >>>>>>>> I need help from the community in determining owners for each >> >>>>>>>> component. >> >>>>>>>> Feel free to directly edit the PR (if you have permission) or >> >>>>>>>> add a comment. >> >>>>>>>> >> >>>>>>>> >> >>>>>>>> Background >> >>>>>>>> The idea is to: >> >>>>>>>> 1. Document good review candidates for each component. >> >>>>>>>> 2. Help choose reviewers using the auto-assignment mechanism. >> >>>>>>>> The suggestion is in no way binding. >> >>>>>>>> >> >>>>>>>> >> >>>> >> >> -- >> Jean-Baptiste Onofré >> jbono...@apache.org >> http://blog.nanthrax.net >> Talend - http://www.talend.com >> >
smime.p7s
Description: S/MIME Cryptographic Signature