Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pre-commit-hooks: remove language_version #144

Closed
wants to merge 1 commit into from
Closed

pre-commit-hooks: remove language_version #144

wants to merge 1 commit into from

Conversation

fsouza
Copy link

@fsouza fsouza commented Jan 31, 2022

This is the same as psf/black#2430.

@asottile
Copy link
Contributor

this value is still correct, I don't really agree with the premise of psf/black#2430

@asottile asottile closed this Jan 31, 2022
@fsouza fsouza deleted the remove-language_version branch January 31, 2022 18:41
adamchainz pushed a commit that referenced this pull request Aug 16, 2023
This enables `default_language_version` stanzas
in consuming `.pre-commit-config.yaml` files to take precedence.

This change is the same as psf/black#2430,
see that PR for full context on why this is beneficial.

This was previously PR'ed to this project as
#144.
It was closed at the time so my company has been keeping a fork of this
project with the patch applied.
With the new maintainership, I wanted to ask whether this change would
be accepted at this time.
Thanks in advance for taking a look, happy to answer any questions.
adamchainz pushed a commit to maxb2/blacken-docs that referenced this pull request Aug 16, 2023
This enables `default_language_version` stanzas
in consuming `.pre-commit-config.yaml` files to take precedence.

This change is the same as psf/black#2430,
see that PR for full context on why this is beneficial.

This was previously PR'ed to this project as
adamchainz#144.
It was closed at the time so my company has been keeping a fork of this
project with the patch applied.
With the new maintainership, I wanted to ask whether this change would
be accepted at this time.
Thanks in advance for taking a look, happy to answer any questions.
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.

None yet

2 participants