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: Fixed some minor spelling errors. #27

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

Conversation

ckaran
Copy link

@ckaran ckaran commented Dec 9, 2020

There are a few things to note about this commit:

First, the corrections use American English spellings. This affects things like 'cancelled' (British) vs. 'canceled' (American). I don't know which dictionary is being used by the project to catch spelling errors, so you may wish to revert those changes.

Second, there are words in use like 'deschedule' that aren't actually in the English language or any of its dialects. However, there is no better word than that, so even if your dictionary flags it in the future, I'd suggest that you keep using it (the word makes sense to native English speakers).

Finally, there are a few grammar errors that I've avoided changing as they are not spelling errors, and therefore outside of the scope of this commit.

There are a few things to note about this commit:

First, the corrections use American english spellings.  This affects
things like 'cancelled' (British) vs. 'canceled' (American).  I don't
know which dictionary is being used by the project to catch spelling
errors, so you may wish to revert those changes.

Second, there are words in use like 'deschedule' that aren't actually in
the English language or any of its dialects.  However, there is no
better word than that, so even if your dictionary flags it in the
future, I'd suggest that you keep using it (the word makes sense to
native English speakers).

Finally, there are a few grammar errors that I've avoided changing as
they are not spelling errors, and therefore outside of the scope of this
commit.
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