The style checker should point out uses of non-inclusive terminology in patches, specifically blacklist/whitelist, master/slave.
(In reply to Simon Fraser (smfr) from comment #0) > The style checker should point out uses of non-inclusive terminology in > patches, specifically blacklist/whitelist, master/slave. Removing 'slave' references requires transitioning to buildbot 0.9, if I recall (although Aakash is expert here) Removing 'master' references is not something I think we can reasonably track in this bug, as newer versions of buildbot still use it to refer to the 'build-master'. Any generalized git tooling will also use 'master' to refer to the default branch since that's what the vast majority of projects use to refer to their primary branch.
Let's start with blacklist/whitelist.
Created attachment 428859 [details] Patch
Style EWS failure in https://ews-build.webkit.org/#/builders/6/builds/47628 is expected (and shows that this code is working).
Comment on attachment 428859 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=428859&action=review > Tools/Scripts/webkitpy/style/checkers/inclusive_language.py:37 > + 'Line contains non-inclusive term.') Can we print the term considered non-inclusive here too?
Created attachment 428867 [details] Patch
(In reply to Jonathan Bedard from comment #5) > Can we print the term considered non-inclusive here too? Done in updated patch.
Committed r277612 (237827@main): <https://commits.webkit.org/237827@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 428867 [details].
<rdar://problem/78127249>