<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 4, 2019 at 1:40 PM Matt Pusateri <<a href="mailto:mpusater@redhat.com">mpusater@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 4, 2019 at 1:15 PM Mike DePaulo <<a href="mailto:mikedep333@redhat.com" target="_blank">mikedep333@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Tue, Jun 4, 2019 at 12:09 PM Robin Chan <<a href="mailto:rchan@redhat.com" target="_blank">rchan@redhat.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Mike, clarification question below...</div><div><br></div><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 4, 2019 at 11:45 AM Mike DePaulo <<a href="mailto:mikedep333@redhat.com" target="_blank">mikedep333@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Tue, Jun 4, 2019 at 9:14 AM Brian Herring <<a href="mailto:bherring@redhat.com" target="_blank">bherring@redhat.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">> Moreover, using black would effectively take style comments/noise out of PR reviews and we could just focus on logic.<div><br></div><div>QE looked into this for a while, and I think it is a WONDERFUL addition... especially for the reasons stated above.<div><br></div><div>Some tweaking would be needed, but it is one less item to worry about.</div><div><br></div><div>+1</div></div></div></blockquote><div>[...]</div><div><br></div><div>+1 based on my experiences on the X2Go project.</div><div>We don't want to turn off new contributors with style issues.<br></div></div></div></blockquote><div>Meaning new contributors would be annoyed that PR reviews would be full of stylistic comments and that would be a bad experience?<br></div><div>I read this as possibly meaning new contributors would be unhappy with the black project style choices? Being unfamiliar with the X2Go project, I'd like some clarification on your input to the discussion here. TIA.</div></div></div></div></blockquote><div><br></div><div>The former. New contributors being annoyed by stylistic comments.</div><div>I think they would likely be unhappy with the black project style choices as well, but that would be outweighed.</div></div><div dir="ltr" class="gmail-m_3097577037529475599gmail-m_-6693824252117726883gmail-m_-5013611176465681213gmail_signature"><div dir="ltr"></div></div></div><br></blockquote><div><br></div><div>This makes me wonder.</div><div><br></div><div>1. What is the ratio of external(non-redhat) committer/contributors?  Meaning if 90% of the code comes from RH, is this really a problem?  I'm not saying that RH should flex its muscle and influence the community, but I am wondering if we're over-thinking it?  Is this a real problem or a perceived problem. <br></div><div>2. Wouldn't you argue that enforcing style guides is the "right thing to do" because it improves code and quality in the long run?  And it theory should also make it easier for new contributors to get involved as there is consistency of code style due the guidelines?</div><div><br></div></div></div></div></blockquote><div><br></div><div>There may be some confusion here. Black doesn't just tell us when something is wrong, it fixes formatting for you. So we have a situation where the code is more consistent *and* easier to contribute. (Unless it does something crazy in our particular case.) <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div></div><div>Matt P. <br></div><div><br></div><div> </div></div></div>
</div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div></div>