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

Add several spelling corrections #844

Merged
merged 1 commit into from Feb 28, 2019

Conversation

Projects
None yet
3 participants
@pabs3
Copy link
Collaborator

pabs3 commented Feb 27, 2019

Also fix some confusion with American vs International English spelling.

@pabs3 pabs3 force-pushed the pabs3:master branch from f13fea0 to 4083df1 Feb 27, 2019

@pabs3

This comment has been minimized.

Copy link
Collaborator Author

pabs3 commented Feb 27, 2019

@pabs3

This comment has been minimized.

Copy link
Collaborator Author

pabs3 commented Feb 27, 2019

@pabs3 pabs3 force-pushed the pabs3:master branch from 4083df1 to 43ea823 Feb 27, 2019

@pabs3

This comment has been minimized.

Copy link
Collaborator Author

pabs3 commented Feb 27, 2019

@larsoner

This comment has been minimized.

Copy link
Member

larsoner commented Feb 27, 2019

Now Travis says

AssertionError: error diabler: correction disabler is an error itself

So if disabler is to be taken as valid, the correction for it should be removed

@pabs3

This comment has been minimized.

Copy link
Collaborator Author

pabs3 commented Feb 27, 2019

@larsoner

This comment has been minimized.

Copy link
Member

larsoner commented Feb 27, 2019

Yeah I'm okay with removing disabler as an error

Add several spelling corrections
Fix some confusion with American vs International English spelling.

Remove a valid word from the misspellings.

@pabs3 pabs3 force-pushed the pabs3:master branch from 43ea823 to 10d8cef Feb 28, 2019

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Feb 28, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #844   +/-   ##
=======================================
  Coverage   91.33%   91.33%           
=======================================
  Files           4        4           
  Lines         727      727           
  Branches      106      106           
=======================================
  Hits          664      664           
  Misses         46       46           
  Partials       17       17

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 2108456...10d8cef. Read the comment docs.

@larsoner larsoner merged commit 4e4dc63 into codespell-project:master Feb 28, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.