Eva Herrada
Eva Herrada
@tannewt, @siddacious and I were just discussing this and I think that there is a very valid reason to add this. Because pylint sets the line length limit at 100,...
@tannewt Yeah, I believe @sommersoft has got a pretty good workflow for patching the build.yml.
@tannewt I was thinking that we shouldn't re-wrap lines already wrapped at 88 characters, since that would be hard to do any other way than by hand, but that it...
I don't think so, but I can definitely test this in a few libraries.
@siddacious thanks for reminding me. I'll take a look today
@tannewt Yep, black will complain if we do that. However, I think since all the black disables that need to be added have already been added, and every PR merged...
@tannewt Oh, just realized I should also mention that black will change things it turned into multiline things to single line things if it is now allowed. Essentially, anything in-between...
@sommersoft Would there be any way to actually run black (not black --check) on all the libraries and then have it change the directive using adabot?