-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 <dal...@redhat.com> wrote: > +0 > > On Mon, Jun 17, 2019 at 1:15 PM Brian Bouterse <bbout...@redhat.com> > wrote: > >> +1 to adopting this. Thank you @daviddavis for writing >> >> On Mon, Jun 10, 2019 at 1:58 PM David Davis <davidda...@redhat.com> >> 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 <gmbno...@gmail.com> 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 >>> Pulp-dev@redhat.com >>> https://www.redhat.com/mailman/listinfo/pulp-dev >>> >> _______________________________________________ >> Pulp-dev mailing list >> Pulp-dev@redhat.com >> https://www.redhat.com/mailman/listinfo/pulp-dev >> > _______________________________________________ > Pulp-dev mailing list > Pulp-dev@redhat.com > https://www.redhat.com/mailman/listinfo/pulp-dev >
_______________________________________________ Pulp-dev mailing list Pulp-dev@redhat.com https://www.redhat.com/mailman/listinfo/pulp-dev