Skip to content

Add a Codespell action#100

Open
peternewman wants to merge 3 commits into
iterative:mainfrom
peternewman:codespell
Open

Add a Codespell action#100
peternewman wants to merge 3 commits into
iterative:mainfrom
peternewman:codespell

Conversation

@peternewman

@peternewman peternewman commented Aug 29, 2022

Copy link
Copy Markdown

Although looking at your other actions, maybe you'd prefer pre-commit which we also support?

To make this clean either fo or ideally that specific line instance needs ignoring too.

@sourcery-ai

This comment was marked as off-topic.

@casperdcl casperdcl added the technical-debt Refactoring, linting & tidying label Oct 26, 2022

@casperdcl casperdcl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Would prefer a pre-commit version.
  • How does the pre-commit version work? Does it require an internet connection or is the dictionary/logic on the local machine?

@pmhahn

pmhahn commented May 7, 2026

Copy link
Copy Markdown

Would prefer a pre-commit version.

pre-commit hook integration also available: https://github.com/codespell-project/codespell

How does the pre-commit version work? Does it require an internet connection or is the dictionary/logic on the local machine?

codespell has its own internal dictionary and works offline — once you have it installed locally.

Comment thread shtab/__init__.py
# - https://opensource.com/article/18/3/creating-bash-completion-script
# - https://stackoverflow.com/questions/12933362
return Template("""\
# AUTOMATICALLY GENERATED by `shtab`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already fixed by 97449e3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

technical-debt Refactoring, linting & tidying

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants