+1 Agree with some of the concerns listed above and in PUP-8. The focus on the logical changes to code and the behavior/quality of the software over code style seems like the correct call.
>From a QE perspective, for how and where to rollout, I would also consider making changes first to test code to minimize changes at once and ensure there are no red herrings added to our process. BRIAN HERRING QUALITY ENGINEER - PULP QE Red Hat <https://www.redhat.com/> 100 East Davie Street Raleigh, NC, 27601 [email protected] M: +19193238427 IM: bherring <https://red.ht/sig> On Tue, Jun 18, 2019 at 8:55 AM Kersom <[email protected]> wrote: > +1 to adopt black code style. > > Reasons already listed. > > > > > > On Tue, Jun 18, 2019 at 4:34 AM Tatiana Tereshchenko <[email protected]> > wrote: > >> -0 >> >> I'm usually all for consistency, and having standard style sounds good in >> theory. >> >> 1. What worries me is that there is basically no way back, we can't just >> try it out. >> Maybe waiting a bit to see if black has more adoption in the Python >> community and goes GA is not a bad idea. >> >> 2. Is now the point when we desperately need autoformatter? >> 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. >> I didn't encounter a noticeable difference in style which will make me >> feel that I need to adopt to it. >> >> 3. If one of the goals is to make the code more readable, in my >> subjective opinion, after black changes, it's 50/50. >> Some parts are more readable, some parts are less. (just to be clear, I'm >> NOT talking here about single or double quotes.) >> >> Tanya >> >> On Tue, Jun 18, 2019 at 4:43 AM Daniel Alley <[email protected]> wrote: >> >>> +0 >>> >>> On Mon, Jun 17, 2019 at 1:15 PM Brian Bouterse <[email protected]> >>> wrote: >>> >>>> +1 to adopting this. Thank you @daviddavis for writing >>>> >>>> On Mon, Jun 10, 2019 at 1:58 PM David Davis <[email protected]> >>>> wrote: >>>> >>>>> 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. >>>>> >>>>> [0] https://github.com/pulp/pups/pull/17 >>>>> [1] https://github.com/pulp/pulpcore/pull/170 >>>>> [2] https://github.com/pulp/pups/blob/master/pup-0001.md#voting >>>>> >>>>> David >>>>> >>>>> >>>>> On Tue, Jun 4, 2019 at 2:53 PM Simon Baatz <[email protected]> wrote: >>>>> >>>>>> On Tue, Jun 04, 2019 at 08:25:47AM -0400, David Davis wrote: >>>>>> > I wanted to get feedback from the Pulp community on using >>>>>> black[0] to >>>>>> > auto-format our Pulp 3 code. I have some mixed feelings about it >>>>>> as I >>>>>> > see some potential benefits of using it but also some downsides >>>>>> as >>>>>> > well. >>>>>> >>>>>> +1 for black (pulp_cookbook uses black for a couple of months now). >>>>>> >>>>> _______________________________________________ >>>>> Pulp-dev mailing list >>>>> [email protected] >>>>> https://www.redhat.com/mailman/listinfo/pulp-dev >>>>> >>>> _______________________________________________ >>>> Pulp-dev mailing list >>>> [email protected] >>>> https://www.redhat.com/mailman/listinfo/pulp-dev >>>> >>> _______________________________________________ >>> Pulp-dev mailing list >>> [email protected] >>> https://www.redhat.com/mailman/listinfo/pulp-dev >>> >> _______________________________________________ >> Pulp-dev mailing list >> [email protected] >> https://www.redhat.com/mailman/listinfo/pulp-dev >> > _______________________________________________ > Pulp-dev mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/pulp-dev >
_______________________________________________ Pulp-dev mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-dev
