On Sun, 25 Jun 2017 at 20:56 Nick Coghlan <ncoghlan at gmail.com> wrote: > On 26 June 2017 at 02:39, Brett Cannon <brett at python.org> wrote: > > On Sat, Jun 24, 2017, 23:45 Nick Coghlan, <ncoghlan at gmail.com> wrote: > >> I was mainly referring to cases where either an initial commit (with a > >> complete NEWS entry) requires follow-up fixes (no NEWS edit > >> necessary). > >> > >> However, another case did occur to me, which is fixing bugs reported > >> against never-released features (which is what I'm currently working > >> on for PEP 538). > > > > https://github.com/python/bedevere/pull/22 is the current > implementation of > > the check. All it does is see if the PR has a path that contains a news > > file. It can also easily be changed to also support some "no news" label > if > > people want it to. > > Yeah, I think I'd prefer to see the current "trivial" label split into > the more explicit "no issue needed" and "no NEWS needed". It isn't > necessarily the case that the changes in such PRs are trivial, it's > that the usual checks don't apply for some reason (and the change > being genuinely trivial is only one possibility) > I've opened https://github.com/python/bedevere/issues/25 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). -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.python.org/pipermail/python-dev/attachments/20170626/f7d5b457/attachment.html>
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4