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

Standard pre-commit config #211

Merged
merged 2 commits into from Jan 16, 2023
Merged

Standard pre-commit config #211

merged 2 commits into from Jan 16, 2023

Conversation

adamchainz
Copy link
Owner

@adamchainz adamchainz commented Jan 16, 2023

The same config I use in all my projects. This ends up formatting the project with Black!

@adamchainz adamchainz changed the title Standard pre-commit Standard pre-commit config Jan 16, 2023
@adamchainz adamchainz merged commit 3cf8b9a into main Jan 16, 2023
@adamchainz adamchainz deleted the standard_pre_commit branch January 16, 2023 10:31
@kurtmckee
Copy link

Just found this project. Thanks for your work on this!

Regarding this PR, I see that blacken-docs (in this repo) is now using the asottile/blacken-docs pre-commit hook. I'm curious if this should instead point to this repo's pre-commit hook?

@adamchainz
Copy link
Owner Author

Since the repo was transferred, GitHub has a redirect, so either URL works to point to this repo. But yes I should update my projects to use the new URL. Thanks for the catch.

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