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

[CLEANUP] Replace a racially-loaded word #1337

Merged
merged 1 commit into from
Sep 24, 2024
Merged

[CLEANUP] Replace a racially-loaded word #1337

merged 1 commit into from
Sep 24, 2024

Conversation

JakeQZ
Copy link
Contributor

@JakeQZ JakeQZ commented Sep 23, 2024

"whitelist" -> "allowlist" in README and a DocBlock comment.

@JakeQZ
[CLEANUP] Replace a racially-loaded word
81390a0 
"whitelist" -> "allowlist" in README and a DocBlock comment.
@JakeQZ JakeQZ added this to the 8.0.0 milestone Sep 23, 2024
@JakeQZ JakeQZ self-assigned this Sep 23, 2024
@coveralls
Copy link

Coverage Status

coverage: 96.841%. remained the same
when pulling 81390a0 on cleanup/allowlist
into 642d2c7 on main.

oliverklee
oliverklee approved these changes Sep 24, 2024
View reviewed changes
Copy link
Contributor

@oliverklee oliverklee left a comment

Choose a reason for hiding this comment

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

❤️

@oliverklee oliverklee merged commit 634155c into main Sep 24, 2024
24 checks passed
@oliverklee oliverklee deleted the cleanup/allowlist branch September 24, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliverklee oliverklee oliverklee approved these changes

Assignees

@JakeQZ JakeQZ

Labels
cleanup documentation enhancement
Projects
None yet
Milestone
8.0.0
Development

Successfully merging this pull request may close these issues.
3 participants
@JakeQZ @coveralls @oliverklee