RESOLVED FIXED 220109
Remove non-inclusive language from JetStream 2.0
https://bugs.webkit.org/show_bug.cgi?id=220109
Summary Remove non-inclusive language from JetStream 2.0
Beth Dakin
Reported 2020-12-22 16:45:00 PST
Running 'report-non-inclusive-language --verbose' inside Websites/browserbench.org/JetStream2.0/ finds a number of non-inclusive terms that we should remove.
Attachments
Patch (1.33 MB, patch)
2020-12-22 16:49 PST, Beth Dakin
no flags
Patch (2.00 MB, patch)
2020-12-23 21:15 PST, Beth Dakin
sam: review+
Patch (1.33 MB, patch)
2021-01-02 15:00 PST, Beth Dakin
no flags
Patch (1.33 MB, patch)
2021-01-02 15:05 PST, Beth Dakin
no flags
Patch (1.33 MB, patch)
2021-01-02 15:10 PST, Beth Dakin
no flags
Patch (1.33 MB, patch)
2021-01-02 15:11 PST, Beth Dakin
no flags
Beth Dakin
Comment 1 2020-12-22 16:49:04 PST
Created attachment 416698 [details] Patch Here's an attempt. I'm not sure how to tell if this breaks things other than running JetStream locally. It ran with and without this change, and the result was similar but slightly better with the change.
Beth Dakin
Comment 2 2020-12-23 21:15:30 PST
Created attachment 416738 [details] Patch Not sure why this patch is so angry, trying again.
Sam Weinig
Comment 3 2020-12-25 10:42:46 PST
Comment on attachment 416738 [details] Patch rs=me! (Assuming the bots get less angry eventually). Thanks for continuing to push this!
Radar WebKit Bug Importer
Comment 4 2020-12-29 16:45:59 PST
Beth Dakin
Comment 5 2021-01-02 15:00:15 PST
Created attachment 416901 [details] Patch Thanks Sam! Trying again.
Beth Dakin
Comment 6 2021-01-02 15:05:59 PST
Beth Dakin
Comment 7 2021-01-02 15:10:07 PST
Beth Dakin
Comment 8 2021-01-02 15:11:48 PST
EWS
Comment 9 2021-01-03 10:43:52 PST
Committed r271126: <https://trac.webkit.org/changeset/271126> All reviewed patches have been landed. Closing bug and clearing flags on attachment 416904 [details].
Note You need to log in before you can comment on or make changes to this bug.