<div dir="ltr">The windows team is notified because the PR includes changes to PCBuild/* </div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">Mariatta Wijaya</div></div>
<br><div class="gmail_quote">On Fri, Oct 6, 2017 at 8:51 AM, Paul Moore <span dir="ltr"><<a href="mailto:p.f.moore@gmail.com" target="_blank">p.f.moore@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hmm, as an example, #2858, which seems to be about the AST (which I'm<br>
not familiar with). I don't particularly want to single this out as a<br>
problem, but it's an example of the sort of request that confuses me -<br>
I simply don't know what help I can offer. Maybe there is some<br>
suspicion that there might be a Windows element - but without some<br>
guidance, I'm not sure where to look.<br>
<span class="HOEnZb"><font color="#888888"><br>
Paul<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 6 October 2017 at 16:38, Zachary Ware <<a href="mailto:zachary.ware%2Bpydev@gmail.com">zachary.ware+pydev@gmail.com</a>> wrote:<br>
> On Fri, Oct 6, 2017 at 7:16 AM, Paul Moore <<a href="mailto:p.f.moore@gmail.com">p.f.moore@gmail.com</a>> wrote:<br>
>> I'm seeing a lot of review requests from github, asking for reviews<br>
>> from the Windows team. Many of the PRs don't as far as I can see have<br>
>> much Windows-specific about them. It doesn't bother me too much (I<br>
>> just ignore ones I don't have anything to say on) but I thought the<br>
>> idea of having the teams was to ask for specific experts to take a<br>
>> look when needed?<br>
>><br>
>> As I say, it's not a big deal for me, but I'm curious how others think<br>
>> the review teams should be used.<br>
><br>
> Do you have some examples of superfluous requests?  I don't think I've<br>
> seen any, other than a rash of bad drive-by PRs that merge a<br>
> maintenance branch into master, which GitHub should be working on<br>
> preventing.  See <a href="https://github.com/python/core-workflow/issues/168" rel="noreferrer" target="_blank">https://github.com/python/<wbr>core-workflow/issues/168</a><br>
> for more on that.<br>
><br>
> --<br>
> Zach<br>
> ______________________________<wbr>_________________<br>
> python-committers mailing list<br>
> <a href="mailto:python-committers@python.org">python-committers@python.org</a><br>
> <a href="https://mail.python.org/mailman/listinfo/python-committers" rel="noreferrer" target="_blank">https://mail.python.org/<wbr>mailman/listinfo/python-<wbr>committers</a><br>
> Code of Conduct: <a href="https://www.python.org/psf/codeofconduct/" rel="noreferrer" target="_blank">https://www.python.org/psf/<wbr>codeofconduct/</a><br>
______________________________<wbr>_________________<br>
python-committers mailing list<br>
<a href="mailto:python-committers@python.org">python-committers@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-committers" rel="noreferrer" target="_blank">https://mail.python.org/<wbr>mailman/listinfo/python-<wbr>committers</a><br>
Code of Conduct: <a href="https://www.python.org/psf/codeofconduct/" rel="noreferrer" target="_blank">https://www.python.org/psf/<wbr>codeofconduct/</a><br>
</div></div></blockquote></div><br></div>