<div dir="ltr"><div>-0</div><div><br></div><div>I'm usually all for consistency, and having standard style sounds good in theory.</div><div><br></div><div>1. What worries me is that there is basically no way back, we can't just try it out. <br></div><div>Maybe waiting a bit to see if black has more adoption in the Python community and goes GA is not a bad idea.</div><div><br></div><div>2. Is now the point when we desperately need autoformatter?<br></div><div> I don't work with many plugins but I read/review code a lot, including pulpcore, pulpcore-plugin, pulp_file, pulp_rpm, pulp_maven, pulp_ansible and a bit of pulp_docker.</div><div> I didn't encounter a noticeable difference in style which will make me feel that I need to adopt to it.<br></div><div><br></div><div>3. If one of the goals is to make the code more readable, in my subjective opinion, after black changes, it's 50/50. <br></div><div>Some parts are more readable, some parts are less. (just to be clear, I'm NOT talking here about single or double quotes.)</div><div><br></div><div>Tanya<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 18, 2019 at 4:43 AM Daniel Alley <<a href="mailto:dalley@redhat.com">dalley@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">+0<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 17, 2019 at 1:15 PM Brian Bouterse <<a href="mailto:bbouters@redhat.com" target="_blank">bbouters@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">+1 to adopting this. Thank you @daviddavis for writing<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 10, 2019 at 1:58 PM David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@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">I opened PUP-8 that proposes adopting black and pydocstyle[0] along with a PR against pulpcore to demonstrate how it would change pulpcore's code. Please review and respond with votes[2]. The deadline will be June 22, 2019.<br class="gmail-m_512688841981164509gmail-m_-4147859512371184737gmail-m_-8877982646805342954gmail-Apple-interchange-newline"><div><br></div><div>[0] <a href="https://github.com/pulp/pups/pull/17" target="_blank">https://github.com/pulp/pups/pull/17</a><br></div><div>[1] <a href="https://github.com/pulp/pulpcore/pull/170" target="_blank">https://github.com/pulp/pulpcore/pull/170</a><br></div><div>[2] <a href="https://github.com/pulp/pups/blob/master/pup-0001.md#voting" target="_blank">https://github.com/pulp/pups/blob/master/pup-0001.md#voting</a></div><div><div><div dir="ltr" class="gmail-m_512688841981164509gmail-m_-4147859512371184737gmail-m_-8877982646805342954gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 4, 2019 at 2:53 PM Simon Baatz <<a href="mailto:gmbnomis@gmail.com" target="_blank">gmbnomis@gmail.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">On Tue, Jun 04, 2019 at 08:25:47AM -0400, David Davis wrote:<br>
>    I wanted to get feedback from the Pulp community on using black[0] to<br>
>    auto-format our Pulp 3 code. I have some mixed feelings about it as I<br>
>    see some potential benefits of using it but also some downsides as<br>
>    well.<br>
<br>
+1 for black (pulp_cookbook uses black for a couple of months now).<br>
</blockquote></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>
_______________________________________________<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>
_______________________________________________<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>