-
Notifications
You must be signed in to change notification settings - Fork 485
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
Typos from the OCaml language #795
Conversation
codespell_lib/data/dictionary.txt
Outdated
@@ -9890,6 +9916,7 @@ perpertrated->perpetrated | |||
perphas->perhaps | |||
perpindicular->perpendicular | |||
perserverance->perseverance | |||
persevered->preserved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
persevered
is the past tense of persevere
so not sure we should add this one
OK. Line deleted.
Not sure why it's failing the CI |
You should check the log https://travis-ci.org/codespell-project/codespell/jobs/478779250#L735 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments from me.
@@ -1844,6 +1847,7 @@ brach->branch | |||
brackeds->brackets | |||
brackground->background | |||
bradcast->broadcast | |||
brakpoint->breakpoint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could also add the plural of this (breakpoints).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
@@ -3720,6 +3726,7 @@ debians->Debian's | |||
debth->depth | |||
debths->depths | |||
debufs->debugfs | |||
debugee->debuggee |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure this is a real word, I assume you mean the person who did the debugging?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e.g. check https://en.wiktionary.org/wiki/debuggee
@@ -5096,6 +5106,8 @@ encosing->enclosing | |||
encosure->enclosure | |||
encounterd->encountered | |||
encouraing->encouraging | |||
encouter->encounter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the plural.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
codespell_lib/data/dictionary.txt
Outdated
@@ -6004,6 +6020,7 @@ foriegn->foreign | |||
forld->fold | |||
forlder->folder | |||
forlders->folders | |||
forma->format, from, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this due to https://en.wikipedia.org/wiki/Pro_forma .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, removed.
codespell_lib/data/dictionary.txt
Outdated
@@ -9469,6 +9493,7 @@ othwerwise->otherwise | |||
othwhise->otherwise | |||
otiginal->original | |||
otion->option | |||
otional->optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add notional as a suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
codespell_lib/data/dictionary.txt
Outdated
@@ -9719,6 +9744,7 @@ parrent->parent | |||
parseing->parsing | |||
parsering->parsing | |||
parsin->parsing | |||
parstree->parsetree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't that be "parse tree"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, corrected.
@@ -12897,6 +12925,7 @@ sucessor->successor | |||
sucessot->successor | |||
sucide->suicide | |||
sucidial->suicidal | |||
sudmobules->submodules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the singular.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
codespell_lib/data/dictionary.txt
Outdated
@@ -13401,6 +13430,7 @@ theres->there's | |||
therfore->therefore | |||
theri->their, there, | |||
thes->this, these, | |||
theses->these |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add thesis as a suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
@@ -13808,6 +13838,7 @@ tyhe->they | |||
tyies->tries | |||
tyope->type | |||
typcial->typical | |||
typecheking->typechecking |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add typechek->typecheck
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
codespell_lib/data/dictionary.txt
Outdated
@@ -4086,6 +4096,7 @@ desccription->description | |||
descencing->descending | |||
descendands->descendants | |||
descendend->descended, descendent, descendant, | |||
descendents->descendants | |||
descentences->descendants, descendents, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Travis complains about the inconsistency here, that descendents
is listed as an error and a correction
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right.
Line descendents->descendants deleted.
Codecov Report
@@ Coverage Diff @@
## master #795 +/- ##
=======================================
Coverage 90.92% 90.92%
=======================================
Files 4 4
Lines 727 727
Branches 106 106
=======================================
Hits 661 661
Misses 48 48
Partials 18 18 Continue to review full report at Codecov.
|
codespell_lib/data/dictionary.txt
Outdated
@@ -13401,6 +13431,7 @@ theres->there's | |||
therfore->therefore | |||
theri->their, there, | |||
thes->this, these, | |||
theses->these, thesis, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
theses
is the plural of thesis
so should be allowed as a correction
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we normally remove them if the incorrect word is actually valid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sometimes yes, sometimes no. Depends on the likely balance of false alarms and misses
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That should probably be another bit in the bitmask:
#410 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks the same as the other cases we were talking about there, so I still only see a need for 0/1 so far
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, maybe I was thinking of "disabled due to" instead.
Some typos from:
https://github.com/ocaml/ocaml/pull/1280/files