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

Update 008-Modern-JS-Architecture.md #51

Closed
wants to merge 1 commit into from
Closed

Conversation

dariye
Copy link
Contributor

@dariye dariye commented Jan 3, 2019

  • Remove huntress as it is gendered. If still needed, warrioress could be used. I would recommend gender inclusivity with the languages we use.
  • Fix typo

- Remove `huntress` as it is gendered. If still needed, [`warrioress`](https://www.merriam-webster.com/dictionary/warrioress) could be used. I would recommend gender inclusivity with the languages we use.
- Fix typo
@MrBenJ
Copy link
Owner

MrBenJ commented Jan 3, 2019

Thanks! Yes, that's a good call. I'm surprised alex didn't catch that... Huh...

I actually have another branch with some heavy edits to this chapter. chapter/architecture. I'll keep your edits in mind while I continue refining this chapter. Going to keep this PR open for now and when I put my edits in the branch, I'll close. Thank you Paul!

@dariye
Copy link
Contributor Author

dariye commented Jan 3, 2019

Awesome! I've completed my first read! Looking forward to the updates

MrBenJ added a commit that referenced this pull request Mar 22, 2019
@MrBenJ MrBenJ closed this Mar 22, 2019
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.

None yet

2 participants