Repository navigation
Commit cc02e45
ci: remove codespell checks and configuration
<!-- Byron -->
Reason: it triggered quite often, is an external dependency
that isn't crucial, and... spelling issues aren't a problem
that GitPython really needs to solve.
rubber-stamp, after checking diff quickly.
<!-- agent -->
Remove `codespell` from the project's pre-commit checks so it no longer
runs locally or in the shared CI lint workflow. This also removes the
need to maintain spelling exceptions for intentional test inputs.
Delete its hook-specific `tomli` dependency, the entire `[tool.codespell]`
configuration and word exception list, and the obsolete inline suppression
on the `remot` archive-option regression case.1 parent f2dfa9f commit cc02e45
3 files changed
Lines changed: 1 addition & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,13 +1,6 @@ | |||
| 1 | 1 | exclude: ^(?:gitdb|smmap)/ | |
| 2 | 2 | ||
| 3 | 3 | repos: | |
| 4 | - - repo: https://github.com/codespell-project/codespell | ||
| 5 | - rev: v2.4.3 | ||
| 6 | - hooks: | ||
| 7 | - - id: codespell | ||
| 8 | - additional_dependencies: [tomli] | ||
| 9 | - exclude: ^test/fixtures/ | ||
| 10 | - | ||
| 11 | 4 | - repo: https://github.com/astral-sh/ruff-pre-commit | |
| 12 | 5 | rev: v0.16.5 | |
| 13 | 6 | hooks: | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -153,9 +153,3 @@ lint.unfixable = [ | |||
| 153 | 153 | "fuzzing/fuzz-targets/**" = [ | |
| 154 | 154 | "E402", # environment setup must happen before the `git` module is imported, thus cannot happen at top of file | |
| 155 | 155 | ] | |
| 156 | - | ||
| 157 | - | ||
| 158 | - [tool.codespell] | ||
| 159 | - ignore-words-list="afile,assertIn,doesnt,gud,uptodate" | ||
| 160 | - #count = true | ||
| 161 | - quiet-level = 3 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1677,7 +1677,7 @@ def test_ignored_raises_error_w_symlink(self): | |||
| 1677 | 1677 | {"rem": "ext::helper"}, | |
| 1678 | 1678 | {"re": "ext::helper"}, | |
| 1679 | 1679 | {"remo": "ext::helper"}, | |
| 1680 | - {"remot": "ext::helper"}, # codespell:ignore remot | ||
| 1680 | + {"remot": "ext::helper"}, | ||
| 1681 | 1681 | {"remote": ["https://example.com/repo", "ext::helper"]}, | |
| 1682 | 1682 | {"remote": (None, False, "ext::helper")}, | |
| 1683 | 1683 | {"remote=ext::helper": True}, | |
0 commit comments