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

Fix typos discovered by codespell #1364

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Aug 6, 2024

Fix typos discovered by codespell -- https://github.com/codespell-project/codespell

codespell --ignore-words-list=bloc,blocs,caf,capela,datas,egal,readd,rin,seh --skip="*.pdf" \
          --quiet-level=3 --write-changes

Copy link

sonarcloud bot commented Aug 6, 2024

@pedrolcl
Copy link
Contributor

pedrolcl commented Aug 6, 2024

codepsell

I see what you did there 😄

@cclauss cclauss changed the title Fix typos discovered by codepsell Fix typos discovered by codespell Aug 6, 2024
Copy link
Member

@derselbst derselbst left a comment

Choose a reason for hiding this comment

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

Thanks!

Codespell is great. I still keep dreaming of a containerized Github action that executes it monthly and opens a PR... never found time for that though.

@derselbst derselbst merged commit 2e60f94 into FluidSynth:master Aug 6, 2024
51 of 54 checks passed
@cclauss cclauss deleted the codespell branch August 6, 2024 20:44
@cclauss
Copy link
Contributor Author

cclauss commented Aug 6, 2024

a containerized Github action that executes it monthly

https://github.com/codespell-project/actions-codespell

Do you use https://pre-commit.com and https://pre-commit.ci?

@derselbst
Copy link
Member

Oh, easier than thought, thanks for the pointer! And no, currently, we don't use any pre commit hooks or similar.

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