Bug 218096 - report-non-inclusive-language should also report total for all the non-inclusive terms
Summary: report-non-inclusive-language should also report total for all the non-inclus...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Aakash Jain
URL:
Keywords: InRadar
Depends on:
Blocks: 213092
  Show dependency tree / graph
 
Reported: 2020-10-22 12:32 PDT by Aakash Jain
Modified: 2021-05-17 10:01 PDT (History)
6 users (show)

See Also:

Attachments
Patch (1.32 KB, patch)
2020-10-22 12:34 PDT, Aakash Jain 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2020-10-22 12:32:48 PDT 
report-non-inclusive-language should also report total usage of all the non-inclusive terms combined. The total can be tracked as a metric and we can target to reduce the total for all the non-inclusive terms.
Comment 1 Aakash Jain 2020-10-22 12:34:59 PDT 
Created attachment 412121 [details]
Patch
Comment 2 Aakash Jain 2020-10-22 12:38:15 PDT 
Also updated the output formatting to make it one-line.

Output before this patch:
[Tools]$report-non-inclusive-language 
total blacklist: 11
total whitelist: 34
total slave: 158


Output after this patch:
[Tools]$report-non-inclusive-language 
Total: 203 (blacklist: 11, whitelist: 34, slave: 158)
Comment 3 EWS 2020-10-22 13:32:58 PDT 
Committed r268890: <https://trac.webkit.org/changeset/268890>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 412121 [details].
Comment 4 Radar WebKit Bug Importer 2020-10-22 13:33:50 PDT 
<rdar://problem/70587102>