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

Typos from the OCaml language #795

Merged
merged 6 commits into from
Jan 17, 2019
Merged

Typos from the OCaml language #795

merged 6 commits into from
Jan 17, 2019

Conversation

Fourchaux
Copy link
Contributor

@@ -9890,6 +9916,7 @@ perpertrated->perpetrated
perphas->perhaps
perpindicular->perpendicular
perserverance->perseverance
persevered->preserved
Copy link
Collaborator

@luzpaz luzpaz Jan 12, 2019

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.
@luzpaz
Copy link
Collaborator

luzpaz commented Jan 12, 2019

Not sure why it's failing the CI

@larsoner
Copy link
Member

Copy link
Collaborator

@peternewman peternewman left a 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
Copy link
Collaborator

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).

Copy link
Contributor Author

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
Copy link
Collaborator

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -5096,6 +5106,8 @@ encosing->enclosing
encosure->enclosure
encounterd->encountered
encouraing->encouraging
encouter->encounter
Copy link
Collaborator

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

@@ -6004,6 +6020,7 @@ foriegn->foreign
forld->fold
forlder->folder
forlders->folders
forma->format, from,
Copy link
Collaborator

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 .

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, removed.

@@ -9469,6 +9493,7 @@ othwerwise->otherwise
othwhise->otherwise
otiginal->original
otion->option
otional->optional
Copy link
Collaborator

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

@@ -9719,6 +9744,7 @@ parrent->parent
parseing->parsing
parsering->parsing
parsin->parsing
parstree->parsetree
Copy link
Collaborator

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"?

Copy link
Contributor Author

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add the singular.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

@@ -13401,6 +13430,7 @@ theres->there's
therfore->therefore
theri->their, there,
thes->this, these,
theses->these
Copy link
Collaborator

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.

Copy link
Contributor Author

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add typechek->typecheck

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

Copy link
Collaborator

@peternewman peternewman left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -4086,6 +4096,7 @@ desccription->description
descencing->descending
descendands->descendants
descendend->descended, descendent, descendant,
descendents->descendants
descentences->descendants, descendents,
Copy link
Member

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

Copy link
Contributor Author

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-io
Copy link

codecov-io commented Jan 16, 2019

Codecov Report

Merging #795 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c241fa...f09a043. Read the comment docs.

@@ -13401,6 +13431,7 @@ theres->there's
therfore->therefore
theri->their, there,
thes->this, these,
theses->these, thesis,
Copy link
Member

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

Copy link
Collaborator

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?

Copy link
Member

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

Copy link
Collaborator

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)

Copy link
Member

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

Copy link
Collaborator

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.

@larsoner larsoner merged commit ee2cc71 into codespell-project:master Jan 17, 2019
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.

5 participants