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

Corrected spelling on architecture / part 1 to 5 #3

Merged
merged 1 commit into from
Oct 24, 2020

Conversation

Cosmeeeen
Copy link
Contributor

In response to this issue, corrected the spelling across all 5 documents using an adaptive spell checker, that also checks for context and manually making changes in order to confirm the right spelling. Would really appreciate a hacktoberfest-accepted if possible, otherwise I'm glad to help. 😄

@preslavmihaylov
Copy link
Owner

@Cosmeeeen the repo has the hacktoberfest topic attached to it, I believe that is sufficient?

I'll still add the label though, just in case

Copy link
Owner

@preslavmihaylov preslavmihaylov left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this!

@preslavmihaylov
Copy link
Owner

@Cosmeeeen could you share what spellchecker you are using? I will consider integrating that with my writing workflow in the future :)

@preslavmihaylov preslavmihaylov merged commit 315bf88 into preslavmihaylov:master Oct 24, 2020
@Cosmeeeen
Copy link
Contributor Author

@Cosmeeeen could you share what spellchecker you are using? I will consider integrating that with my writing workflow in the future :)

Sure! I used Grammarly, which is a free browser extension + any online text editor. It has some errors on more complex grammar, and it isn't really adapted for more scientific texts, but if you manually correct every highlighted error you can get a really good result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants