-
Notifications
You must be signed in to change notification settings - Fork 931
Fix typos and spelling errors #2990
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
Conversation
|
@sinapis-amicus thank you very much for taking the time to find, and also to correct, these spelling mistakes! I have left one comment regarding one of the changes you have suggested, can you let me know what you think? Also, can you update the commit message for the commit to include |
|
@sinapis-amicus actually, I won't be able to push the changes back to this PR branch, since you haven't created the PR from a branch on your fork, but rather directly from your develop branch. The contributing document explains the process that we like to see folks following when suggesting changes to this repository. Since I have already made them changes locally, including rebasing the 3 commits here into a single PR, I will go ahead and merge that directly, and close this PR. Your contributions will still be correctly attributed to you, it is just that the commits won't be coming from this PR. Thanks again for your help in fixing these spelling mistakes, appreciate it! |
|
This PR has been rebased and merged into the develop branch here: f2b0590 |
Thanks Gary. Happy New Year |
You are very welcome! And to you! |
|
🎉 This issue has been resolved in version 2.0.0 🎉 The release is available on: Your GitReleaseManager bot 📦🚀 |
(doc)
Description Of Changes
Fixed some minor typos and spelling errors.
Motivation and Context
Improve code quality and documentation legibility.