openstack-doc-core team mailing list archive
-
openstack-doc-core team
-
Mailing list archive
-
Message #00493
Re: Review Rigour
On 17/11/16 12:55, Anne Gentle wrote:
>
>
> On Tue, Nov 8, 2016 at 5:42 AM, Alexandra Settle <alexandra.settle@xxxxxxxxxxxxx <mailto:alexandra.settle@xxxxxxxxxxxxx>> wrote:
>
<snip>
>
> Sorry for all the questions! Just many thoughts running through my head. Let it be known that I definitely think this is a good idea! But I suggest some lines are drawn so we are all clearly on the same page.
>
>
> I suggest we try it and see what chaos ensues with these guidelines:
> 1. If it's technically accurate, merge it. If it fixes a bug correctly, merge it.
> 2. If it's accurate and correct but could be written better, edit, then merge it with a comment to coach the person how the writing could be better.
> 3. If it's not the kind of patch we want for the docs, explain that in the review and also follow up to make sure the person doesn't feel rejected outright. Take ownership of the coaching areas more than the "this is wrong and here's why" aspect. (I'm not saying your reviews are like that, mind you, I just want ownership of the growth of contributors and the accurate doc base, not ownership of "it meets our English standards.")
>
I like these guidelines!
What about relaxing our requirements on the number of votes required? Is a +1 and a +2A enough?
L
--
Lana Brindley
Technical Writer
Rackspace Cloud Builders Australia
http://lanabrindley.com
Attachment:
signature.asc
Description: OpenPGP digital signature
Follow ups
References