[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

Replace instances of "blacklist/whitelist" with "denylist/allowlist" #366

Merged
merged 2 commits into from
Jun 15, 2020
Merged

Replace instances of "blacklist/whitelist" with "denylist/allowlist" #366

merged 2 commits into from
Jun 15, 2020

Conversation

morgante
Copy link
Contributor
@morgante morgante commented Jun 13, 2020 •
edited
Loading

Fixes #362

@morgante morgante force-pushed the fix/inclusive-language branch 2 times, most recently from ebdf84f to fc10cb9 Compare June 13, 2020 06:20
@morgante morgante changed the title [WIP] Replace instances of "blacklist/whitelist" with "denylist/allowlist" Replace instances of "blacklist/whitelist" with "denylist/allowlist" Jun 13, 2020
@morgante morgante marked this pull request as ready for review June 13, 2020 06:26
gkowalski-google
gkowalski-google approved these changes Jun 15, 2020
View reviewed changes
Copy link
Contributor
@gkowalski-google gkowalski-google left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for this change!

@morgante morgante merged commit 3d8a9ce into master Jun 15, 2020
@morgante morgante deleted the fix/inclusive-language branch July 8, 2020 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@briantkennedy briantkennedy briantkennedy approved these changes

@gkowalski-google gkowalski-google gkowalski-google approved these changes

@AdrienWalkowiak AdrienWalkowiak Awaiting requested review from AdrienWalkowiak

@t12g t12g Awaiting requested review from t12g

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Change to more inclusive language
4 participants
@morgante @briantkennedy @gkowalski-google @googlebot