mirror of
https://github.com/Stirling-Tools/Stirling-PDF.git
synced 2026-09-03 05:10:16 +03:00
James Brunton
eb08e60d67
# Description of Changes The `pre-commit` commands in this repo are inconsistent with the rest of the dev workflow, as they are impossible to run through Task and they can cause CI to fail with no way for a developer to run the `pre-commit` scripts after they've failed. This PR adds `task pre-commit` (and `task pre-commit:fix`) and then hooks up the existing `pre-commit` hooks and CI to call the Task rule, so if developers are using pre-commit hooks then they should still work, but they're also runnable without using pre-commit at all. I think it'd be worth reviewing what we're actually running at pre-commit in the future because I'm not entirely convinced by all of the scripts that we are running, but this should at least make what we have properly enforced and usable by all devs.
61 lines
1.8 KiB
Python
61 lines
1.8 KiB
Python
#!/usr/bin/env python3
|
|
"""Trailing-whitespace and end-of-file normalisation, driven by Task.
|
|
|
|
Replaces the end-of-file-fixer / trailing-whitespace pre-commit hooks, which
|
|
have no read-only mode. Run via `task pre-commit` (check) and `task
|
|
pre-commit:fix`; Task selects the files (with `git ls-files`) and passes them
|
|
as arguments.
|
|
|
|
python scripts/whitespace.py <files>... # check: report, exit 1 if any need fixing
|
|
python scripts/whitespace.py --fix <files>... # fix: rewrite in place
|
|
|
|
Operates on bytes and only ever touches trailing spaces/tabs and the final
|
|
newline, so it never mangles content or line endings. Binary files (those with
|
|
a NUL byte) are skipped.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
|
|
def normalise(data: bytes) -> bytes:
|
|
# Strip trailing spaces/tabs from each line (leave \r so CRLF survives).
|
|
lines = [line.rstrip(b" \t") for line in data.split(b"\n")]
|
|
body = b"\n".join(lines)
|
|
# Ensure a non-empty file ends with exactly one newline.
|
|
stripped = body.rstrip(b"\r\n")
|
|
return stripped + b"\n" if stripped else body
|
|
|
|
|
|
def main() -> int:
|
|
args = sys.argv[1:]
|
|
fix = "--fix" in args
|
|
paths = [a for a in args if a != "--fix"]
|
|
|
|
offenders: list[str] = []
|
|
for path in paths:
|
|
data = Path(path).read_bytes()
|
|
if b"\0" in data:
|
|
continue
|
|
fixed = normalise(data)
|
|
if fixed == data:
|
|
continue
|
|
offenders.append(path)
|
|
if fix:
|
|
Path(path).write_bytes(fixed)
|
|
|
|
if offenders and not fix:
|
|
print(f"{len(offenders)} file(s) need whitespace fixing:")
|
|
for path in offenders:
|
|
print(f" {path}")
|
|
return 1
|
|
if offenders and fix:
|
|
print(f"Fixed whitespace in {len(offenders)} file(s).")
|
|
return 0
|
|
|
|
|
|
if __name__ == "__main__":
|
|
sys.exit(main())
|