Black: remove py36

Created on 8 Feb 2019  路  6Comments  路  Source: psf/black

Hi,

With --target-version pushed, what's the point of keeping py36 for black ?
Considering it's been removed in blackd, it should be the same for black.

Most helpful comment

I've developed a strong opinion :) let's remove it

All 6 comments

I didn't want to drop backward compatibility with existing black configurations.

Also, I don't think I removed support for anything from blackd. What feature do you think was removed?

oh sorry I read to fast the implementation.
But since since black hates configuration, I think this should be at least deprecate or deprecation warning in a next release.
After all it's still in beta, it's still time for deprecating some useless options.

FWIW I also think we should drop --py36 while we're in pre-release. There should be one way to do things, and we shouldn't be afraid to deprecate features/APIs in this phase.

Sounds good. Should we just drop it in the upcoming release or leave it in for one more for compatibility?

I don't have a strong opinion on it, but if I were to do it, I'd remove it in the next release :)

I've developed a strong opinion :) let's remove it

Was this page helpful?
0 / 5 - 0 ratings

Related issues

underyx picture underyx  路  22Comments

dgingrich picture dgingrich  路  36Comments

kindjacket picture kindjacket  路  21Comments

ambv picture ambv  路  51Comments

rollschild picture rollschild  路  35Comments