codespell-project / codespell

check code for common misspellings
GNU General Public License v2.0
1.9k stars 466 forks source link

Codespell Not Ignoring Words Properly when Provided in Original Case #3385

Closed SOHAN-cyber closed 7 months ago

SOHAN-cyber commented 7 months ago

Codespell currently fails to properly ignore words if they are provided in their original case in the ignore file. For instance, when the word "OptIn" is added to the ignore file exactly as it is, Codespell does not recognize it as an ignored word during spell checks. However, when the word is added in lowercase, such as "optin", Codespell successfully ignores it as intended.

Pre-commit Hook Config

  - repo: https://github.com/codespell-project/codespell
    rev: v2.2.6
    hooks:
      - id: codespell
        additional_dependencies:
          - tomli
        args:
          - --ignore-words=extern/make/.config/.codespellignore
          - --config=extern/make/.config/.codespell.toml

.codespellignore

OptIn

Error:

CodeSpell Issue

Solution

To ensure Codespell ignores words correctly, users are currently required to add them in lowercase to the ignore file.

Updated .codespellignore File:

optin

You can find the proposed solution in the following link:

https://cobalt.googlesource.com/cobalt/+/master/.pre-commit-config.yaml#:~:text=%23%20The%20%2D%2Dignore%2Dwords,be%20lowercase%2C%20see

DimitriPapadopoulos commented 7 months ago

Codespell currently fails to properly ignore words if they are provided in their original case in the ignore file.

I think this is not true, see #3272. Besides, I don't understand how a .pre-commit-config.yaml could help.