search

bigmlcom / sensenet

0 stars 8 forks source link

Blacken #23

Closed charleslparker closed 3 years ago

charleslparker commented 3 years ago

This looks like a lot, but I'm just adding black for formatting, harmonizing this with many of the other python projects we're starting these days. My code style and black's already pretty much agree, except that I use(d) single quotes, and prefer a shorter line length. There should be no substantiative changes here.

jimshur commented 3 years ago

This looks like a lot, but I'm just adding black for formatting, harmonizing this with many of the other python projects we're starting these days. My code style and black's already pretty much agree, except that I use(d) single quotes, and prefer a shorter line length. There should be no substantiative changes here.

FWIW, in mltoolbox and accushoot (vandura) the one black default we're overriding is for the line length (we're not barbarians!). We compromised on 88 characters wide, which is still fine for Github reviews, and allows me to have two full-width pages of code side-by-side on my laptop.