Re: [PATCH v2 0/3] Remove use of "whitelist"
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jul 15, 2022, 06:30 UTC
- Message-ID
- <xmqq5yjy3n2c.fsf@gitster.g>
- In-Reply-To
- <pull.1274.v2.git.1657852722.gitgitgadget@gmail.com>
"Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 17 quoted lines
> The word "whitelist" has cultural implications that make its use > non-inclusive. > > A previous version of this series recommended the replacement of > "allowlist", but that term is still new and not accepted by some common > dictionaries. > > Instead, this version avoids the use of "whitelist" by rewording the > sentences that use it. In many cases, this improves readability since the > term is used suddenly without other context (and in some cases, is not > necessary at all). > > There is one case where "whitelist" is replaced with "allow_list" but that > is because we are operating on a string list parsed from the > GIT_ALLOW_PROTOCOL environment variable. > > Thanks, -Stolee
Maybe I am biased, but for all the changes in these patches, I find the updated text far easier to understand than a mere replacing of the words s/white/allow/, even if I pretend that allowlist is considered by everybody a proper part of English vocabulary. After all, I think most of the places did not have to say "whitelist" in the first place.
Will queue.
Thanks.