-
Notifications
You must be signed in to change notification settings - Fork 112
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 has changed from hooks.yaml -> .pre-commit-hooks.yaml #16
Comments
In 0.16.0, pre-commit has stopped reading the legacy `hooks.yaml` file and is instead reading only `.pre-commit-hooks.yaml`. The following issues track getting this updated / fixed: - guykisel/pre-commit-robotframework-tidy#1 - hootsuite/pre-commit-php#16 - pricematch/mirrors-closure-linter#1 - elidupuis/mirrors-standard#2 - jordant/rubocop-pre-commit-hook#4 - sanmai-NL/pre-commit-hooks_R#1 - magicmark/pre-commit-es6-imports-reorder#1
This repository has been temporarily removed from pre-commit.com as the latest version of pre-commit (0.16.0) has removed support for legacy |
We use these plugins at our company all day long, so I've forked the repo to digitalpulp/pre-commit-php. I'll pull @djun-kim #21 into that repo and push it up in a moment. |
@FatherShawn sweet, I'll add that repo for the time being as this one seems to be no longer maintained |
@asottile Thanks! We are super grateful to Yelp for pre-commit and to Hootsuite for contributing th e PHP hooks. And we are happy to take them over since this issue has been open for so long. |
👍 there's probably one or more of these PRs you'll want to incorporate as well: https://github.com/hootsuite/pre-commit-php/pulls |
closing due to inactivity |
Hello pre-commit hook implementer!
In version 0.12.0 pre-commit has changed the default location for the file formerly known as
hooks.yaml
to make it more convincing for others to add more hooks.As such, a migration has to (unfortunately) occur.
For maximum compatibility it is suggested to
cp hooks.yaml .pre-commit-hooks.yaml
(at least for the migration period). A copy is suggested over a symlink unless you do not care for windows compatibility (and I wouldn't blame you!).Once the migration period is over (or you no longer care to support old versions of pre-commit), the hooks.yaml file is no longer necessary and may be deleted.
See pre-commit/pre-commit#470 for more details
Thanks again for contributing to the pre-commit ecosystem, we couldn't do it without you :)
Anthony
The text was updated successfully, but these errors were encountered: