[Openstack-docs] Setting WIP by core-reviewers?

Anne Gentle annegentle at justwriteclick.com
Thu Dec 5 01:03:35 UTC 2013


If other core teams are reviewing in this way I'm fine with it too.

Anne


On Wed, Dec 4, 2013 at 6:50 PM, Jeremy Stanley <fungi at yuggoth.org> wrote:

> On 2013-12-04 21:05:12 +0100 (+0100), Andreas Jaeger wrote:
> > During the review of creating openstack-doc-tools, I was asked whether
> > we want to have the ability to allow core-reviewers to set
> > Workinprogress on a patch.
> [...]
>
> Just a tip: if your core review team is struggling with an
> overwhelming volume of outstanding changes, having the ability to
> acknowledge that a contributor will definitely need to do more work
> on one by setting it to WIP rather than merely expressing an opinion
> on its readiness can be a great convenience to fellow reviewers. It
> allows you to politely send the message that it needs additional
> work while simultaneously removing it from the list of ready changes
> until that happens. (Of course, you should still explain to the
> contributor what needs doing; that goes without saying.)
> --
> Jeremy Stanley
>
> _______________________________________________
> Openstack-docs mailing list
> Openstack-docs at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs
>



-- 
Anne Gentle
annegentle at justwriteclick.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-docs/attachments/20131204/76d6c3c4/attachment-0001.html>


More information about the Openstack-docs mailing list