Bug 225827

Summary: report-non-inclusive-language should skip node_modules directory
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, ap, bdakin, darin, ryanhaddad, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 213092    
Attachments:
Description Flags
Patch none

Description Aakash Jain 2021-05-14 15:02:26 PDT
node_modules contain external code. Some of the modules might contain non-inclusive language. report-non-inclusive-language script should skip node_modules.

e.g.: 
https://github.com/WebKit/WebKit/tree/main/PerformanceTests/Speedometer/resources/todomvc/architecture-examples/angularjs/node_modules
https://github.com/WebKit/WebKit/tree/main/Websites/browserbench.org/Speedometer2.0/resources/todomvc/architecture-examples/angularjs/node_modules
Comment 1 Aakash Jain 2021-05-14 15:04:02 PDT
Created attachment 428667 [details]
Patch
Comment 2 EWS 2021-05-15 04:00:54 PDT
Committed r277537 (237765@main): <https://commits.webkit.org/237765@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 428667 [details].
Comment 3 Radar WebKit Bug Importer 2021-05-15 04:01:22 PDT
<rdar://problem/78054158>