Request for two new MR-labels: "review-needed" and "changes-required"
Andreas Klebinger
klebinger.andreas at gmx.at
Fri Feb 7 16:42:25 UTC 2020
Hello devs,
Recently in a MR discussion the topic came up of contributors being
frustrated about lack of reviews.
As a contributor and as a reviewer the lack of tools to find MR's in
need of attention also frustrated me often.
I suggest two new MR labels:
Label: "review-needed": If a contributor has an MR which he considers
to be blocked on (lack of) review he can assign the label to clearly
signal this.
Label: "changes-required": This is the inverse - if a reviewer looked at
a patch and it's clear that the patch needs more work from the
contributor this label can signal this.
What are the benefits?
* As a reviewer I can look for MR's tagged with review-needed and can be
sure that time spent reviewing is not wasted.
* As a contributor I can mark a patch as changes-required if I think my
patch is ready to land, or if I need input on the design.
* Core maintainers can more easily identify patches which have been
stalled for a long time, making their review a priority.
The obvious downside is more labels.
Cheers
Andreas
More information about the ghc-devs
mailing list