-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider adding eslint-plugin-inclusive-language #380
Comments
We could add this, but it's relatively more of an issue on the PHP side of the fence, I feel. The additional processing cost of using this would be pretty trivial, however, so I'm not against. |
It would probably be better implemented as a code-language-agnostic tool like https://github.com/wikimedia/grunt-tyops, otherwise we'll need to re-implement in stylelint, phplint etc. |
Should we just add these as default 'typos' to that repo, then? |
Ideally yes, but there are many places where blacklist/whitelist still need to be mentioned as public or deprecated APIs, or references to 3rd party APIs, and can't be removed for some time. |
Would help with making our code more inclusive and the effort in https://phabricator.wikimedia.org/T279275
Thoughts?
ES: https://github.com/muenzpraeger/eslint-plugin-inclusive-language#readme
The text was updated successfully, but these errors were encountered: