Adjust 'black' code to support version 18 as well #943
Conversation
|
Hi @keith-packard..! Thank you for the contribution. Just so we get some context here, why does Mu need to support version 18 when version 19 is already released..? |
Debian stable ships version 18, and I need to use that release. With this change, mu runs great. Note that setup.py requires version 18, not 19, so this change is required to actually work when that is the version available. |
keith-packard
force-pushed
the
black-version
branch
from
a8f6ad4
to
30f8ec0
Compare
|
This pull request introduces 1 alert when merging 30f8ec0 into ee600ff - view on LGTM.com new alerts:
|
keith-packard
force-pushed
the
black-version
branch
from
30f8ec0
to
c4065bf
Compare
keith-packard
force-pushed
the
black-version
branch
from
c4065bf
to
6c898c0
Compare
Version 18 of 'black' has a slightly different API; check which version we've managed to load and adapt on the fly Signed-off-by: Keith Packard <keithp@keithp.com> -- v2: reformat code to fit in 79 columns fix coverage tests v3: pass exceptions not related to black version through avoid importing all of 'black'
keith-packard
force-pushed
the
black-version
branch
from
6c898c0
to
8bb791e
Compare
|
Sorry Keith! I missed this PR when creating #1519. I'll close this one the other has already been merged. Thanks again for the contribution though! |
Version 18 of 'black' has a slightly different API; check which
version we've managed to load and adapt on the fly
Signed-off-by: Keith Packard keithp@keithp.com