<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"Intel Clear Light";
panose-1:2 11 4 4 2 2 3 2 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman",serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.hoenzb
{mso-style-name:hoenzb;}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
font-family:"Calibri",sans-serif;}
@page WordSection1
{size:8.5in 11.0in;
margin:70.85pt 70.85pt 70.85pt 70.85pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D">+1 to approach suggested by sdake.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D">Furthermore, I think it would be good if -1/0/+1 only reflects logical meaning of reviewed docs while still providing some suggestions on improving spelling and
grammar in comment even if we +1 given patch.<o:p></o:p></span></p>
<p class="MsoNormal"><i><span style="font-size:11.0pt;font-family:"Intel Clear Light",sans-serif;color:#1F497D"><br>
Alicja </span></i><span style="font-size:11.0pt;font-family:"Intel Clear Light",sans-serif;color:#1F497D"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><a name="_____replyseparator"></a><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Martin André [mailto:martin.andre@gmail.com]
<br>
<b>Sent:</b> Wednesday, April 13, 2016 12:03 PM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions) <openstack-dev@lists.openstack.org><br>
<b>Subject:</b> Re: [openstack-dev] [kolla][vote] Nit-picking documentation changes<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">On Tue, Apr 12, 2016 at 10:05 PM, Steve Gordon <<a href="mailto:sgordon@redhat.com" target="_blank">sgordon@redhat.com</a>> wrote:<o:p></o:p></p>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt">----- Original Message -----<br>
> From: "Jeff Peeler" <<a href="mailto:jpeeler@redhat.com">jpeeler@redhat.com</a>><br>
> To: "OpenStack Development Mailing List (not for usage questions)" <<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
><br>
> On Mon, Apr 11, 2016 at 3:37 AM, Steven Dake (stdake) <<a href="mailto:stdake@cisco.com">stdake@cisco.com</a>><br>
> wrote:<br>
> > Hey folks,<br>
> ><br>
> > The reviewers in Kolla tend to nit-pick the quickstart guide to death<br>
> > during<br>
> > reviews. I'd like to keep that high bar in place for the QSG, because it<br>
> > is<br>
> > our most important piece of documentation at present. However, when new<br>
> > contributors see the nitpicking going on in reviews, I think they may get<br>
> > discouraged about writing documentation for other parts of Kolla.<br>
> ><br>
> > I'd prefer if the core reviewers held a lower bar for docs not related to<br>
> > the philosophy or quiickstart guide document. We can always iterate on<br>
> > these new documents (like the operator guide) to improve them and raise the<br>
> > bar on their quality over time, as we have done with the quickstart guide.<br>
> > That way contributors don't feel nitpicked to death and avoid improving the<br>
> > documentation.<br>
> ><br>
> > If you are a core reveiwer and agree with this approach please +1, if not<br>
> > please –1.<br>
><br>
> I'm fine with relaxing the reviews on documentation. However, there's<br>
> a difference between having a missed comma versus the whole patch<br>
> being littered with misspellings. In general in the former scenario I<br>
> try to comment and leave the code review set at 0, hoping the<br>
> contributor fixes it. The danger is that a 0 vote people sometimes<br>
> miss, but it doesn't block progress.<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal">My typical experience with (very) occasional drive by commits to operational project docs (albeit not Kolla) is that the type of nit that comes up is more typically -1 thanks for adding X, can you also add Y and Z. Before you know it a
simple drive by commit to flesh out one area has become an expectation to write an entire chapter.<o:p></o:p></p>
</blockquote>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt">That's because you're a native speaker and you write proper English to begin with :)<br>
<br>
We should be asking ourselves this simple question when reviewing documentation patch "does it make the documentation better?". Often the answer is yes, that's why I'm trying to ask for additional improvements in follow-up patches.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt">Regarding spelling or a grammatical mistakes, why not fix it now while it's still hot when we spot one in the new documentation that's being written? It's more time consuming to fix it later. If needed a native
speaker can take over the patch and correct English.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Martin<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<p class="MsoNormal"><span class="hoenzb"><span style="color:#888888">-Steve</span></span><o:p></o:p></p>
<div>
<div>
<p class="MsoNormal"><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">
OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><o:p></o:p></p>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>