Pre-commit: prevent spaces in news entry paths #126063
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow on to python/core-workflow#186 and https://discuss.python.org/t/new-blurb-1-2-please-upgrade/59159:
When it fails, it tells you what to do, for example:
https://github.com/python/cpython/actions/runs/11548867437/job/32140948311?pr=126063
We may get some old PRs merged that don't have the check, but it will be easy to cleanup later. And we often update old PRs from
mainto re-trigger the CI and see the warning before merge.Thanks to @adamchainz for https://adamj.eu/tech/2024/01/24/pre-commit-fail-hook/