Skip to content

gh-109408: Add the docs whitespace check from patchcheck to pre-commit #109854

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

Merged
hugovk merged 12 commits into from
Oct 10, 2023
Merged

gh-109408: Add the docs whitespace check from patchcheck to pre-commit #109854

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
dff437b
Run pre-commit as part of 'make patchcheck'
hugovk Sep 20, 2023
97c6ccf
Use f-strings
hugovk Sep 20, 2023
029a9a4
Add whitespace
hugovk Sep 20, 2023
d7690a3
Properly pluralise file(s), fix Unix typo
hugovk Sep 20, 2023
5cfcd19
Remove normalize_docs_whitespace from patchcheck, it's covered by pre…
hugovk Sep 20, 2023
1eb8042
Include inc files in pre-commit's trailing-whitespace
hugovk Sep 20, 2023
a85cd1c
Fix string
hugovk Sep 25, 2023
433153d
Use "s" variable for readability
hugovk Sep 25, 2023
a429344
Reinstate docs_modified
hugovk Sep 25, 2023
63c6d17
Reinstate doc_files for docs_modified
hugovk Sep 26, 2023
05a5d35
Revert makefile changes
hugovk Sep 27, 2023
f4b16ec
Merge branch 'main' into patchcheck-docs-whitespace
hugovk 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
Next Next commit
Properly pluralise file(s), fix Unix typo
  • Loading branch information
@hugovk
hugovk committed Sep 25, 2023
commit d7690a3588a7f32117a730c35a2697fa9cc17688
5 changes: 3 additions & 2 deletions Tools/patchcheck/patchcheck.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -281,8 +281,9 @@ def ci(pull_request):
if not fixed:
print('No whitespace issues found')
else:
print(f'Please fix the {len(fixed)} file(s) with whitespace issues')
print('(on UNIX you can run `make patchcheck` to make the fixes)')
count = len(fixed)
print(f'Please fix the {count} {n_files_str(count)} with whitespace issues')
Comment thread
hugovk marked this conversation as resolved.
Outdated
print('(on Unix you can run `make patchcheck` to make the fixes)')
sys.exit(1)


Expand Down