<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Sun, 25 Jun 2017 at 20:56 Nick Coghlan <<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 26 June 2017 at 02:39, Brett Cannon <<a href="mailto:brett@python.org" target="_blank">brett@python.org</a>> wrote:<br>
> On Sat, Jun 24, 2017, 23:45 Nick Coghlan, <<a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>> wrote:<br>
>> I was mainly referring to cases where either an initial commit (with a<br>
>> complete NEWS entry) requires follow-up fixes (no NEWS edit<br>
>> necessary).<br>
>><br>
>> However, another case did occur to me, which is fixing bugs reported<br>
>> against never-released features (which is what I'm currently working<br>
>> on for PEP 538).<br>
><br>
> <a href="https://github.com/python/bedevere/pull/22" rel="noreferrer" target="_blank">https://github.com/python/bedevere/pull/22</a> is the current implementation of<br>
> the check. All it does is see if the PR has a path that contains a news<br>
> file. It can also easily be changed to also support some "no news" label if<br>
> people want it to.<br>
<br>
Yeah, I think I'd prefer to see the current "trivial" label split into<br>
the more explicit "no issue needed" and "no NEWS needed". It isn't<br>
necessarily the case that the changes in such PRs are trivial, it's<br>
that the usual checks don't apply for some reason (and the change<br>
being genuinely trivial is only one possibility)<br></blockquote><div><br></div><div>I've opened <a href="https://github.com/python/bedevere/issues/25">https://github.com/python/bedevere/issues/25</a> to change this for the issue number check (although I will name the label "skipped issue" so all check-skipping labels and start with the same prefix for sorting purpose). <br></div></div></div>