Skip to content

Add a Codespell action#100

Open
peternewman wants to merge 3 commits intoiterative:mainfrom
peternewman:codespell
Open

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

Conversation

@peternewman
Copy link
Copy Markdown

@peternewman peternewman commented Aug 29, 2022

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
Copy link
Copy Markdown
Collaborator

@casperdcl casperdcl left a comment

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
Copy link
Copy Markdown

pmhahn commented May 7, 2026

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