ci.yml: redundant pip install pre-commit #137

New issue

Closed

opened 2026-07-06 06:29:32 +00:00 by the.auditor · 0 comments

the.auditor commented

2026-07-06 06:29:32 +00:00

Owner

.forgejo/workflows/ci.yml:23 runs pip install pre-commit separately, but pre-commit is already a dependency of [dev] (see pyproject.toml:20). The pip install -e ".[dev]" on line 20 already installs it.

Remove line 23 (pip install pre-commit) — it's dead weight.

Ref: PR #136

`.forgejo/workflows/ci.yml:23` runs `pip install pre-commit` separately, but `pre-commit` is already a dependency of `[dev]` (see pyproject.toml:20). The `pip install -e ".[dev]"` on line 20 already installs it. Remove line 23 (`pip install pre-commit`) — it's dead weight. Ref: PR #136

the.auditor added the

Kind/Enhancement

Priority

Low

labels

2026-07-06 06:29:34 +00:00