Skip to content

Conversation

@hugovk
Copy link
Contributor

@hugovk hugovk commented Jan 20, 2020

The --py36 option was deprecated in Black 19.3b0 (#750):

deprecated --py36 (use --target-version=py36 instead) (#724)

https://github.com/psf/black#193b0

It's been a year since it was deprecated, and has been deprecated in two releases (19.3b0 and 19.10b0).

Is it now time to remove the deprecation?

At least I think it would be good to remove before the non-prerelease Black release (#517), and whilst still in beta too.

@hugovk hugovk changed the title Rm deprecated --py36 option Remove deprecated --py36 option Jan 23, 2020
@JelleZijlstra JelleZijlstra self-assigned this May 8, 2020
@JelleZijlstra
Copy link
Collaborator

I agree that it's time for this, but there's a complex merge conflict now (sorry for the wait!).

@hugovk hugovk force-pushed the rm-deprecated--py36 branch from c51c06f to 77c4fc4 Compare May 8, 2020 13:59
@hugovk hugovk force-pushed the rm-deprecated--py36 branch from 77c4fc4 to 36edb84 Compare May 8, 2020 14:40
@hugovk
Copy link
Contributor Author

hugovk commented May 8, 2020

Merge conflict resolved!

@JelleZijlstra JelleZijlstra merged commit 8e38b66 into psf:master May 8, 2020
@hugovk hugovk deleted the rm-deprecated--py36 branch May 8, 2020 14:53
jakubvalenta added a commit to jakubvalenta/blacken that referenced this pull request Apr 5, 2021
Use the option --target-version insteead --py36, which was removed
in Black 20.8b0.

The option --target-version was added in Black 19.3b0, so blacken now
requires at least this version of Black.

See psf/black#1236
dakra pushed a commit to pythonic-emacs/blacken that referenced this pull request Apr 5, 2021
Use the option --target-version insteead --py36, which was removed
in Black 20.8b0.

The option --target-version was added in Black 19.3b0, so blacken now
requires at least this version of Black.

See psf/black#1236
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants