[Webkit-unassigned] [Bug 175670] webkit-patch land should remove r and cq flags

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Aug 17 19:29:02 PDT 2017


Michael Catanzaro <mcatanzaro at igalia.com> changed:

           What    |Removed                     |Added
                 CC|                            |mcatanzaro at igalia.com

--- Comment #2 from Michael Catanzaro <mcatanzaro at igalia.com> ---
(In reply to Alexey Proskuryakov from comment #1)
> Why is this desirable? r+ in particular is a useful part of patch history,
> and I think that commit queue removing it is a mistake.

I would change commit-queue instead. It's nice to see at a glance who reviewed the patch from Bugzilla.

The only harm from flags in closed bugs comes from r? and cq? flags, in which case yes they need to be cleared to remove the patches from request queue. But even that is a bit silly: request queue could just learn to ignore them.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170818/bf4ba0c1/attachment.html>

More information about the webkit-unassigned mailing list