Philipp/template-git
Template
1
0
Fork 0
Code Issues 6 Pull Requests Actions Packages Releases Activity

chore: consolidate redundant pre-commit hooks into ruff #19

New Issue
Open
opened 2026-02-26 01:25:09 +01:00 by Philipp · 0 comments
Philipp commented 2026-02-26 01:25:09 +01:00
Owner
Copy Link

The current .pre-commit-config.yaml contains several hooks that can be replaced or consolidated by ruff, which is already used in the project. Consolidating these hooks reduces overhead and simplifies maintenance.

Redundant hooks that Ruff can handle:

  • check-ast: Ruff parses the AST and will fail on syntax errors.
  • check-docstring-first: Ruff's D rules (e.g., D301) can handle docstring placement.
  • trailing-whitespace: Ruff can handle this via W291 and W293.
  • end-of-file-fixer: Ruff can handle this via W292.

Recommendation:
Evaluate expanding tool.ruff.lint.select to include these rules and subsequently removing the corresponding hooks from pre-commit-hooks.

The current `.pre-commit-config.yaml` contains several hooks that can be replaced or consolidated by `ruff`, which is already used in the project. Consolidating these hooks reduces overhead and simplifies maintenance. Redundant hooks that Ruff can handle: - `check-ast`: Ruff parses the AST and will fail on syntax errors. - `check-docstring-first`: Ruff's `D` rules (e.g., `D301`) can handle docstring placement. - `trailing-whitespace`: Ruff can handle this via `W291` and `W293`. - `end-of-file-fixer`: Ruff can handle this via `W292`. Recommendation: Evaluate expanding `tool.ruff.lint.select` to include these rules and subsequently removing the corresponding hooks from `pre-commit-hooks`.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
Renovate Philipp
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Philipp/template-git#19
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?