-
Notifications
You must be signed in to change notification settings - Fork 251
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
broken pre-commit ci #1476
Comments
You mean like #1477 ? This won't update the pre-commit versions |
The PRs should still be created. The CI will just continue to fail. Is there a way to disable CI but not the PRs by pre-commit bot? |
We just merged the pre-commit check so we can bypass the failing one. Let's wait a week and see if it gets fixed. We had a similar issue last year. Maybe we re evaluate the approach depending on this ? |
Ahg! It is mypy being too slow c602666! |
The pre-commit hok for Add |
yes! try what it works! the idea is to make sure it runs locally but not on a merge commit. I tried to fix it but failed :( |
I know there is an open source one that we can run like any other action
Thoughts @juanitorduz
The text was updated successfully, but these errors were encountered: