Issue metadata
Sign in to add a comment
|
add cq? equivalent to codereview tool |
||||||||||||||||||||||||
Issue descriptionIn WebKit, you could put cq+ to put something in the cq. cq- to tell people not to put things in the cq (e.g. because you know it depends on another patch). And cq? to tell others to put it in the cq if they believe it's ready (typically this would be the reviewe). Being able to express that intent comes up frequently. Can we add something like that to polygerrit?
,
Sep 20 2016
+jochen@ this isn't exactly what you've explained and asked for before, but I think you'd be interested.
,
Sep 20 2016
There'll need to some sort of guard against the combination of "cq+" and "lgtm with nits" to make sure the author gets a chance to fix the nits.
,
Sep 20 2016
This is more or less exactly what I asked for in https://bugs.chromium.org/p/chromium/issues/detail?id=162819
,
Sep 27 2016
,
Apr 5 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by andyb...@chromium.org
, Sep 20 2016