Re: [PATCH v2 0/3] Remove use of "whitelist"
- From
- Phillip Wood <phillip.wood123@gmail.com>
- Date
- Jul 15, 2022, 16:16 UTC
- Message-ID
- <14d7e4f5-e8da-d8f7-05f3-a30faf25f8c8@gmail.com>
- In-Reply-To
- <xmqq5yjy3n2c.fsf@gitster.g>
On 15/07/2022 07:30, Junio C Hamano wrote:
Show 23 quoted lines
> "Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes: > >> 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/,
I was struck by that as well, the new wording is definitely an improvement, thanks for working on it Stolee
Best Wishes
Phillips.