Blacklist -> disallowlist #2
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Cool project. Any chance you’d be open to changing "blacklist" to the more inclusive "disallowlist" (or a similar variant)?
Thanks!
I suspect a simple rename is not what you are proposing? Can you give more information as to why the blacklist isn't doing what it should be doing for you?
Hello, I'd like to add my +1 for this issue as seeing this language made me bristle a bit while configuring my install.
I can't speak for OP, but as far as I can tell theres nothing wrong with the feature as such -- the problem is that the words 'blacklist' and 'whitelist' have racist origins which contribute to making tech less inclusive.
There are dozens of resources availiable online to which explain the issue much more eloquintly than I can.
Here is one such resource that I like: https://inclusivenaming.org/word-lists/tier-1/whitelist/
I'd love to help out on this, would you be open to me submitting a PR which aliases allowlist->whitelist andd denylist->blackliist ?
Oh I see what you mean, I had no idea. Sure, go ahead!
Hi all, I've just submitted a PR for this change (hope that's OK!).
Reading through that 'inclusive naming' list, I also think the following terms should be updated too, if possible?
abort
->terminate[d]
master
->main
(this is currently used for the default branch & referenced in theREADME.md
only)Thanks, I merged your request!
I guess we can keep doing these changes but I'll close this for now as the main offending ones are gone.