-
Notifications
You must be signed in to change notification settings - Fork 20
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, brands and acronyms #21
Conversation
ccoVeille
commented
Apr 1, 2024
- fix typos in code, test and documentation
- Fix registered trademark and other acronyms
Hi @ccoVeille I have moved some functions from Once done, I will merge this PR into master, thanks! |
No problem. I'll when I have a minute later today. |
It should be OK now. |
I recommend using simple tools from time to time, such as https://github.com/crate-ci/typos More information about them in an old article I made on dev.to https://dev.to/ccoveille/how-to-fix-typos-in-your-code-for-goods--3c7 |
Great, I'll integrate them into the git hooks |
I would say this one is the simplest to use |
How do you handle false positives? Like these: https://github.com/xhd2015/xgo/actions/runs/8520319809/job/23336201289?pr=24 I'd like
|
https://github.com/marketplace/actions/typos-action#false-positives add a _typos.toml [default]
extend-ignore-identifiers-re = [
"[Ll]its", # Lits is a valid concept here
"nd", # short variable I would like to use
] |
The file can be named https://github.com/crate-ci/typos/blob/master/docs/reference.md |
Thanks, learned a lot |