← Back to team overview

nagios-charmers team mailing list archive

Re: [Merge] ~addyess/charm-nagios:blacken into charm-nagios:master

 

Review: Disapprove

As ryan noted, the black run concat'ed a few strings, but I see that's because black was set to 120 line-length.  We are going to be running 88 line length blacken runs this week.  Is there anything else you linted other than the black quotes and line length refactoring?

I'm considering dropping this commit.  We'll be hitting lint and black on these charms with new common lint and black configurations.
-- 
https://code.launchpad.net/~addyess/charm-nagios/+git/charm-nagios/+merge/387617
Your team Nagios Charm developers is subscribed to branch charm-nagios:master.


References