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 "blacklist/whitelist" terminology in code comments, and internal variable, method and class names #1484

Closed
tlfeng opened this issue Nov 1, 2021 · 0 comments · Fixed by #2178 or #2784
Closed

Replace "blacklist/whitelist" terminology in code comments, and internal variable, method and class names #1484

tlfeng opened this issue Nov 1, 2021 · 0 comments · Fixed by #2178 or #2784
Labels
enhancement Enhancement or improvement to existing feature or request v1.3.0

Comments

@tlfeng
Copy link
Collaborator

tlfeng commented Nov 1, 2021 •
edited
Loading

Is your feature request related to a problem? Please describe.
Replace "blacklist/whitelist" terminology in the code base, but only the places that won't impact the backwards compatibility.

A part of #1483

Describe the solution you'd like
Replace all the "blacklist/whitelist" terminology with "denylist/allowlist" in the following location:

  • code comments
  • Internal variable, method and class names

Any APIs or configurations will not be impacted.

Describe alternatives you've considered
None.

Additional context
None.
@tlfeng tlfeng added enhancement Enhancement or improvement to existing feature or request untriaged and removed untriaged labels Nov 1, 2021
@tlfeng tlfeng mentioned this issue Nov 1, 2021
Closed
10 tasks
@tlfeng tlfeng mentioned this issue Nov 10, 2021
Closed
@tlfeng tlfeng added the v1.3.0 label Nov 15, 2021
@tlfeng tlfeng linked a pull request Feb 21, 2022 that will close this issue
Replace exclusionary words whitelist and blacklist in the places that won't impact backwards compatibility #2178
Merged
@tlfeng tlfeng mentioned this issue Feb 23, 2022
Merged
@saratvemulapalli saratvemulapalli mentioned this issue Mar 15, 2022
Closed
10 tasks
@tlfeng tlfeng mentioned this issue Apr 6, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Enhancement or improvement to existing feature or request v1.3.0
Projects
None yet
Milestone
No milestone
1 participant
@tlfeng