Skip to content

Adapt/extend codespell config#119

Open
ruestefa wants to merge 35 commits intoMeteoSwiss-APN:mainfrom
ruestefa:codespell-config
Open

Adapt/extend codespell config#119
ruestefa wants to merge 35 commits intoMeteoSwiss-APN:mainfrom
ruestefa:codespell-config

Conversation

@ruestefa
Copy link
Contributor

Suggested changes/extensions to codespell config:

  • Move ignores to file (default entries are up for debate, I just copied the file from my notes repo)
  • Keep ignoring all-uppercase words (acronyms), but not all uppercase words
    • Catch typos at the beginning of sentences at the expense of the occasional false-positive given name
  • Skip lines with codespell-ignore comment (e.g., # codespell-ignore in Bash, Python etc., <!-- codespell-ignore --> in Markdown etc.)
  • Skip files with .codespell-ignore. in name
    • Alternative to hardcoding the file name in the config

clairemerker and others added 29 commits April 5, 2023 18:22
# Conflicts:
#	requirements/requirements.yml
#	tmpl/requirements/requirements.yml.j2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants