ADMIN: Auto-label PRs for review

[dsaxton]

Not certain this will work, but nonetheless I think something like this would be helpful (analogous to "Needs Triage" for issues)

[rhshadrach]

At what point does the "Needs Review" tag get removed?

[dsaxton]

At what point does the "Needs Review" tag get removed?

I think it'd be the judgement of whoever is doing the reviewing, but initial thought is to take it down as soon as initial feedback is given (so the label means something like "not yet looked at"; so you could untag this one 😄). I'm imagining a way of making it easier to do "visual triage" when looking at the PR list (and also to have meaningful exemptions if the linked stale PR action gets merged).

[rhshadrach]

That makes sense. I might even think of going further, to untag if the PR "has legs". That is, if it appears to be going in the right direction, even if more refinement is needed.

[simonjayhawkins]

I'm happy with this in principal since sometimes I look at the PR list for PRs without comments but that misses PRs where the author has added additional comments outside the OP.

[WillAyd]

Sure I see no harm in trying this out

[jreback]

sure