On Mon, 2012-08-27 at 17:59 +0000, Marcin Cieslak wrote:
When "need-review" should be removed?
If I understand it correctly: If the review takes place in Gerrit
anyway, then I don't see much sense in having this keyword in Bugzilla.
It should rather indicate something like "The attached patch needs be
moved to Gerrit so it can get a review in Gerrit".
I see only "patch-reviewed" in the keywords
- which can be
both negative and positive.
Apart from whether it makes sense to have this keyword if patch review
takes place in Gerrit anyway:
"patch-reviewed" obviously refers to a patch, while keywords refer to a
report. Hence using a keyword does not work well if a report includes
more than one patch.
This sounds like a classic usecase for a Bugzilla flag:
Set "review?" for requesting a review for a patch, get "review+" for
acceptance or "review-" for refusal.
andre
--
Andre Klapper (
maemo.org bugmaster & GNOME Bugsquad)
http://blogs.gnome.org/aklapper/