-
Notifications
You must be signed in to change notification settings - Fork 1.8k
fix(NODE-3792): remove offensive language throughout the codebase #3091
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
fix(NODE-3792): remove offensive language throughout the codebase #3091
Conversation
This is ready for review. The build failures are all the Task Timeouts we've discussed. It would be good to get a second set of eyes to ensure I didn't change anything that should be deprecated instead. |
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.
The code all LGTM, really great job here @ljhaywar ! Just waiting on the tests to get sorted out :)
I reran the local patch from my machine with Maybe it was the mistake I made that Neal caught? Or maybe it was unrelated as I've been seeing the Context Cancelled errors sporadically in my builds and others'? Either way, I think we're good to go. |
Description
Remove references to "master" throughout the codebase
What is changing?
Is there new documentation needed for these changes?
No
What is the motivation for this change?
Following the guidance documented in tools.ietf.org/id/draft-knodel-terminology-00.html, we will remove all oppressive and unnecessarily gendered language in driver documentation, code, tests, specs, and spec tests.
Double check the following
npm run check:lint
script<type>(NODE-xxxx)<!>: <description>