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 found by codespell #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix typos found by codespell #21

wants to merge 1 commit into from

Conversation

DimitriPapadopoulos
Copy link

Some typos are in user-visible strings and documentation, others are in code comments.

In the long term, I recommend fixing all typos, including those in code comments, because tools like codespell do not make a difference between code comments and documentation and user-visible strings.

I have left out typos in Contrib, they should be fixed upstream.

@DimitriPapadopoulos
Copy link
Author

DimitriPapadopoulos commented Dec 30, 2021

Not sure about _miscrosoft_docs_are_wrong_microsoft_docs_are_wrong.

I believe it's OK because it's a private variable.

@sredna
Copy link
Collaborator

sredna commented Dec 30, 2021

I have left out typos in Contrib, they should be fixed upstream

We own most of Contrib. If there is anything upstream I'll deal with it.

@DimitriPapadopoulos
Copy link
Author

Fixed typos in Contrib too.

@DimitriPapadopoulos
Copy link
Author

DimitriPapadopoulos commented Dec 31, 2021

I have left out Language files by the way.

There may be a few starting comments in English to fix, for example appropiateappropriate:

; - "Elimină" related terms replaced with more appropiate "Dezinstalează"

Should I fix these comments?

Edit: That's actually the only typo in there; I have fixed it.

@DimitriPapadopoulos
Copy link
Author

DimitriPapadopoulos commented Dec 31, 2021

And I just noticed part of the initial fixes, under Docs/src/halibut, must actually be fixed upstream.

Edit: I've sent a mail to the upstream maintainer.
Edit: Ongoing discussion with the upstream maintainer: are both bulletted and bulleted (as in the Oxford dictionary) correct in UK English?

@sredna
Copy link
Collaborator

sredna commented Dec 31, 2021

are both bulletted and bulleted (as in the Oxford dictionary) correct in UK English?

Bulletted only has 35k hits on Google vs millions for Bulleted. I'll take the fix.

@sredna sredna self-requested a review January 4, 2022 01:20
@DimitriPapadopoulos
Copy link
Author

I see you have merged this change into the Subversion trunk. Thanks! Would it be worth adding a configuration file .codespellrc to silence false positives and simplify future maintenance?

@sredna
Copy link
Collaborator

sredna commented Jan 10, 2022

I suppose that depends on how many false-positives you had to deal with and what the maintenance overhead is...

@DimitriPapadopoulos
Copy link
Author

I am afraid nsis has many false positives:

$ codespell | wc -l
954
$ 

Most of them are in translation files that should be skipped. A .codespellrc file could drastically reduce that number by skipping these files. For example:

$ codespell --skip 'Language files' | wc -l
226
$ 

The can further reduce that number by ignoring some words:

$ codespell --skip 'Language files' --ignore-words-list parms | wc -l
133
$ 

The benefit of a .codespllrc is that developers get consistent results when running codespell locally.

Fix last typos in the process.

About "everytime" see for example:
https://www.grammarly.com/blog/everytime-every-time/
@DimitriPapadopoulos
Copy link
Author

All fixes have been merged, thank you. I have added a .codespellrc file and fixed the last typos. Do you think it's worth merging?

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.

2 participants