-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Cleans up docs & edu material by removing simple/easy/etc words #3241
Conversation
Deploy preview ready! Built with commit 0959c14 |
I’ll code review later today!
On Sat, Dec 16, 2017 at 12:21 gatsbybot ***@***.***> wrote:
Deploy preview ready!
Built with commit 3aee403
<3aee403>
https://deploy-preview-3241--gatsbygram.netlify.com
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3241 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AA4gJ6ZNpykxFA6B8v0XYd4ZZ_hAWnk9ks5tA_wGgaJpZM4REZR_>
.
|
This is fantastic! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey looked through this! Looks great. Just re-added a couple instances of the word "simple" that were part of code and are therefore necessary. Also corrected a misspelled word in the original post.
Thanks @benjaminhoffman and @shannonbux! |
Awesome! I didn't get a chance to check earlier—just want to say thanks for this great change. |
It reads so much better 🎆. Extra thanks from me as someone that finds GraphQL completely confusing. |
By suggestion of #3159 @robwierzbowski
tldr: remove words that may seem intimidating to beginners / starts.
This should handle about 60% of the cleanup.