Gerrit is commonly used as a place to share works in progress early.

This is great, but it has an unfortunate side effect of making it
harder for would-be reviewers to find patches that need reviewing
using this query:
https://gerrit.wikimedia.org/r/#/q/(NOT+label:Verified%253D-1)+AND+(label:Code-Review%253D0+OR+NOT+(label:Code-Review%253D-1+OR+label:Code-Review%253D-2)),n,z

Could I ask that as a norm, if you post a WIP patch that you also self -2 it?

In addition to this, if a patch is open for longer than several months
you may want to abandon it - it's much more useful to link to an
abandoned patchset in a phabricator task which has all the context for
someone who might be able to solve the problem it tries to solve.

I'd love to get us to a place where
https://gerrit.wikimedia.org/r/#/q/(NOT+label:Verified%253D-1)+AND+(label:Code-Review%253D0+OR+NOT+(label:Code-Review%253D-1+OR+label:Code-Review%253D-2)),n,z
is manageable that code gets merged left right and center :)

Warm regards,
Jon

_______________________________________________
Wikitech-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to