<div dir="ltr">If you use a recent version of 'git review,' it accepts a list of reviewers to add via --reviewers.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 1, 2016 at 4:33 PM, Joel Stanley <span dir="ltr"><<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Nov 2, 2016 at 6:04 AM, Brendan Higgins<br>
<<a href="mailto:brendanhiggins@google.com">brendanhiggins@google.com</a>> wrote:<br>
>> I'm not sure how you can enforce specific reviewers in Gerrit without<br>
>> yet another config file per repo that we need to maintain.<br>
><br>
><br>
> Again, the goal of listing reviewers is not to enforce a particular set of<br>
> reviewers, but to offer a quick and easy reference as to who you should be<br>
> talking to; where to direct your questions; and consequently good people to<br>
> put on your code reviews; I see no reason to make that a requirement since a<br>
> maintainer has to sign-off anyway.<br>
<br>
</div></div>Can we add tooling so pushing a review will cc the relevant<br>
maintainers automatically?<br>
<br>
In a past life I used repo to interact with gerrit, and we would add<br>
the reviewers to the command line. This would be similar to the kernel<br>
workflow of using <a href="http://get_maintainers.pl" rel="noreferrer" target="_blank">get_maintainers.pl</a> and git send-email. Perhaps this<br>
is already available for non-repo Gerrit users, and we just need to<br>
add it to our documentation.<br>
<br>
On the proposal in general; I think it's a good move. Did we consider<br>
using something like the OWNERS file that is used by Chromium? In that<br>
project, developers are added to an OWNERS file anywhere in the source<br>
tree to indicate they should be consulted over changes to that part of<br>
the tree.<br>
<br>
 <a href="http://dev.chromium.org/developers/owners-files" rel="noreferrer" target="_blank">http://dev.chromium.org/<wbr>developers/owners-files</a><br>
<br>
As an example, this allows me indicate that I should be consulted when<br>
you are submitting a change to the u-boot or kernel parts of the tree,<br>
such as meta-phosphor/common/recipes-<wbr>kernel.<br>
<br>
Cheers,<br>
<br>
Joel<br>
</blockquote></div><br></div>