Update codespell skip list in pre-commit config#111
Conversation
Added additional paths to the codespell skip list.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the .pre-commit-config.yaml file to expand the list of paths that the codespell hook should skip. The feedback suggests that the path evaluation/pdf2reproschema/old/evaluation is redundant in the skip list, as codespell recursively skips subdirectories and evaluation/pdf2reproschema/old is already included. It is recommended to remove this redundant entry to simplify the configuration.
| - id: codespell | ||
| args: | ||
| - --skip=poetry.lock,docs_style/pdoc-theme/syntax-highlighting.css,*.cha,*.ipynb,example/sample_metadata.csv | ||
| - --skip=poetry.lock,docs_style/pdoc-theme/syntax-highlighting.css,*.cha,*.ipynb,example/sample_metadata.csv,evaluation/benchmark,evaluation/pdf2reproschema/old,evaluation/ner/old,evaluation/pdf2reproschema/old/evaluation |
There was a problem hiding this comment.
The path evaluation/pdf2reproschema/old/evaluation is redundant because it is a subdirectory of evaluation/pdf2reproschema/old, which is already included in the skip list. codespell recursively skips content in specified directories, so the explicit inclusion of the subdirectory is not needed. Removing it will simplify the configuration.
- --skip=poetry.lock,docs_style/pdoc-theme/syntax-highlighting.css,*.cha,*.ipynb,example/sample_metadata.csv,evaluation/benchmark,evaluation/pdf2reproschema/old,evaluation/ner/old
1 participant
Added additional paths to the codespell skip list (evaluation/benchmark,evaluation/pdf2reproschema/old,evaluation/ner/old,evaluation/pdf2reproschema/old/evaluation).