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
Fix pre-commit configuration #6328
Conversation
maintenance: | ||
- '.pre-commit-config.yaml' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can see, the labeler is fixed and should no longer remove lables.
Also pre-commit do not plan to support labels in it configuration pre-commit-ci/issues#24
Codecov Report
@@ Coverage Diff @@
## main #6328 +/- ##
==========================================
- Coverage 91.98% 91.91% -0.07%
==========================================
Files 595 595
Lines 52586 52586
==========================================
- Hits 48371 48337 -34
- Misses 4215 4249 +34 |
Oh, this isn't merged yet; let's get it in! |
# Description In PR #6328 I have added an automatic maintenance label when something is edited pre-commit. But I wrongly understand actions/lableler documentation and thought that the bug was fixed, that is true, but it is fixed in v5 branch, not backported to v4. v5 has been from a long time (since May) in the release process. actions/labeler#112
Description
Stop fixing spaces in vendored modules.
Fix bug pointed in #6318