[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/8] scripts/device-crash-test: Replace the word 'whitelis
|
From: |
Philippe Mathieu-Daudé |
|
Subject: |
Re: [PATCH v2 4/8] scripts/device-crash-test: Replace the word 'whitelist' |
|
Date: |
Wed, 3 Mar 2021 19:50:28 +0100 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.0 |
On 3/3/21 1:06 PM, Alex Bennée wrote:
>
> Philippe Mathieu-Daudé <philmd@redhat.com> writes:
>
>> Follow the inclusive terminology from the "Conscious Language in your
>> Open Source Projects" guidelines [*] and replace the word "whitelist"
>> appropriately.
>>
>> [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md
>>
>> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>> scripts/device-crash-test | 30 +++++++++++++++---------------
>> 1 file changed, 15 insertions(+), 15 deletions(-)
>> def whitelistTestCaseMatch(wl, t):
>> - """Check if a test case specification can match a whitelist entry
>> + """Check if a test case specification can match a allowlist entry
>
> Seems like the function names (and params?) could be updated as well.
Eduardo already fixed/merged this one :)
See 1a14d4e16af ("device-crash-test: Remove problematic language")