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

Inclusive terminology #647

Open
jwstein3400 opened this issue Nov 27, 2020 · 2 comments · May be fixed by #648
Open

Inclusive terminology #647

jwstein3400 opened this issue Nov 27, 2020 · 2 comments · May be fixed by #648

Comments

@jwstein3400
Copy link

Is your feature request related to a problem? Please describe.
In the bandit repository there are currently 27 instances of blacklist, 1 instance of whitelist, and 6 instances of master.

These terms while not intentionally are negative in connotation and should be considered for replacement.

Describe the solution you'd like
I would like an update that results in inclusive terms used in documentation, source code, and in general.
For more context linux kernel merge, email discussion.

Describe alternatives you've considered
Some options to consider:
In place or master (for code branch main):

  • '{primary,main} / {secondary,replica,subordinate}'
  • '{initiator,requester} / {target,responder}'
  • '{controller,host} / {device,worker,proxy}'
  • 'leader / follower'
  • 'director / performer'

In place of black/white list:

  • 'denylist / allowlist'
  • 'blocklist / passlist'
@lukehinds
Copy link
Member

+1

Do you want to make this change, or just let me know and I can.

@jwstein3400
Copy link
Author

jwstein3400 commented Nov 27, 2020

  1. I can submit a change for blacklist/s to blocklist/s.

  2. The whitelist reference is in the tox.ini and would require an upgrade to 3.18 or later.

  3. Of the master references I can change 1 of them, while 3 are associated with the default branch and 2 are Sphinx configurations. Coming soon #conf.py parameter for default document name uses exclusionary language sphinx-doc/sphinx#8326

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 a pull request may close this issue.

2 participants