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

RFC changes 01 #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

robertfchapman
Copy link

While getting familiar with your code I noticed a couple Grammatical/Spelling Errors. Sorry no actual code additions or bug fixes at this time.

@sixfiveohtwo
Copy link

Initialise isn't a spelling error. It's 'British English', or as we like to call it in England, English ;) Anyway, I'm not the developer and I don't know him but I do know that he's English so language choices should be at his discretion. This is just a polite request that you don't refer to the correct use of someone's native language as an error or something that should be fixed.

@robertfchapman
Copy link
Author

@sixfiveohtwo Thanks for 'British English' lesson. I have often come across this and made changes, not realizing that there was a British spelling of "Initialise'. My spell checker doesn't even accept it. I'll have to add it to my spelling dictionary as it is correct English.

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