Problem
The README's “Auto-formatting with pre-commit” section tells contributors to install the pre-commit executable with:
uv tool install pre-commit
It then says every commit will be checked, but it never installs the repository's Git hook. Without that hook, commits are not checked locally and formatting failures are only caught after CI runs.
This occurred on #3004: manual pre-commit run --all-files validation happened while newly added files were still untracked, so they were skipped; the commit itself did not run a hook and CI later caught the Ruff formatting changes.
Suggested change
Add the hook installation step after installing the executable:
It may also be worth clarifying that pre-commit run --all-files only covers Git-tracked files, while the commit hook checks staged files, including newly added files.
Problem
The README's “Auto-formatting with pre-commit” section tells contributors to install the
pre-commitexecutable with:It then says every commit will be checked, but it never installs the repository's Git hook. Without that hook, commits are not checked locally and formatting failures are only caught after CI runs.
This occurred on #3004: manual
pre-commit run --all-filesvalidation happened while newly added files were still untracked, so they were skipped; the commit itself did not run a hook and CI later caught the Ruff formatting changes.Suggested change
Add the hook installation step after installing the executable:
It may also be worth clarifying that
pre-commit run --all-filesonly covers Git-tracked files, while the commit hook checks staged files, including newly added files.