Skip to content

Commit 0ae00fd

Browse files
authored
docs: require pre-commit before every push in CLAUDE.md (#709)
1 parent 7a92eeb commit 0ae00fd

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

CLAUDE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -225,6 +225,7 @@ pytest tests/
225225
- **Each PR should have a single purpose**: Whether it is a bug fix, a new feature, a refactor, or a documentation update, keep it to one thing per PR.
226226
- **Run the full test suite before submitting**: Run `make test` or `pytest` and ensure all tests pass.
227227
- **Ensure mypy and ruff pass**: Run `mypy snap7 tests example` and `ruff check snap7 tests example` with no errors before opening a PR.
228+
- **Always run `uv run pre-commit run --all-files` before every `git push`.** Individual `ruff check` / `ruff format --check` commands don't exercise every hook (`ruff format` is the one that actually reformats files, not the `--check` variant). Skipping pre-commit is the single most common reason CI fails on the ruff-format hook right after a push. If the hook reformats, amend and re-push — do not rely on "it passed locally" via other commands.
228229
- **If using AI coding assistants**: Review the generated code carefully before submitting. AI-generated PRs that are large, unfocused, or not thoroughly reviewed are likely to be rejected.
229230

230231
## Library Architecture Notes

0 commit comments

Comments
 (0)