Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pre-commit: Add codespell hook #38

Merged
merged 1 commit into from
Jul 23, 2024
Merged

pre-commit: Add codespell hook #38

merged 1 commit into from
Jul 23, 2024

Conversation

alexdewar
Copy link
Collaborator

Add codespell pre-commit hook. I've found it useful on a few projects so I think it's probably worth rolling out more widely. It generally doesn't produce too many false positives (it errs on the side of false negatives) and you can always add extra words to .codespell_ignore.txt if it does.

Closes #30.

@alexdewar alexdewar marked this pull request as ready for review July 22, 2024 15:47
@alexdewar alexdewar merged commit f12e3df into main Jul 23, 2024
8 checks passed
@alexdewar alexdewar deleted the codespell branch July 23, 2024 08:09
@AdrianDAlessandro
Copy link
Contributor

Am I interpreting this correctly that this will only apply to this repo and not any repos that are built with this tool? There are two .pre-commit-config.yaml files and this PR only edits the one at the top level and not the one inside the {{ cookiecutter.project_slug }} directory. The same goes for any changes to the workflows, as these appear to be duplicated also.

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.

Add codespell pre-commit hook
3 participants