Skip to content

[3.11] GH-109408: Move the C file whitespace check from patchcheck to pre-commit (GH-109890) #110640

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.

Sign up for GitHub

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

Closed
AA-Turner wants to merge 3 commits into from
+42 −103
Closed

[3.11] GH-109408: Move the C file whitespace check from patchcheck to pre-commit (GH-109890) #110640

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fd8ddd4
[3.11] GH-109408: Move the C file whitespace check from patchcheck to…
AA-Turner Oct 10, 2023
afd9d76
fix the directory
AA-Turner Oct 10, 2023
707ffaa
Fix excludes
AA-Turner Oct 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix excludes
  • Loading branch information
@AA-Turner
AA-Turner committed Oct 10, 2023
commit 707ffaa5b16a8b2e8c5b37bcccb114a6522817fb
5 changes: 3 additions & 2 deletions .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ repos:
entry: 'python Tools/scripts/reindent.py --nobackup --newline LF'
language: 'system'
types: [python]
exclude: '^(Lib/test/tokenizedata/|Tools/c-analyzer/cpython/_parser).*$'
exclude: '^(Lib/test/tokenizedata/|Tools/c-analyzer/cpython/(_parser|_capi)).*$'

- repo: local
hooks:
Expand All @@ -35,7 +35,8 @@ repos:
# Don't check the style of vendored libraries
exclude: |
(?x)^(
Modules/_ctypes/libffi_osx/.*
Modules/_ctypes/darwin/dlfcn.h
| Modules/_ctypes/libffi_osx/.*
| Modules/_ctypes/libffi_msvc/.*
| Modules/_decimal/.*
| Modules/libmpdec/.*
Expand Down