Skip to content
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

Inclusive language #1875

Closed
andrewkantor opened this issue Sep 15, 2020 · 2 comments · Fixed by #1949
Closed

Inclusive language #1875

andrewkantor opened this issue Sep 15, 2020 · 2 comments · Fixed by #1949
Assignees
Labels
bug Bug report

Comments

@andrewkantor
Copy link
Contributor

Can you address a couple of instances of language in the project where the wording can be made more inclusive? (example: whiltelist -> allowlist). We are trying to make the code in all LinkedIn projects more inclusive. Could you please examine whether or not these need to be updated, and make the changes? For suggested replacements see go/inclusivelanguage or google. THANK YOU!

Term URL
Whitelist https://github.com/linkedin/datahub/blob/5072ef013ce3ccb116726fda38fefb7d8319fd11/datahub-web/@datahub/shared/types/configurator/configurator.d.ts
Whitelist https://github.com/linkedin/datahub/blob/0116a955cbc3fe0fdba6e0effecad3c6791f8862/datahub-web/@datahub/shared/addon/components/institutional-memory/containers/tab.ts
Whitelist https://github.com/linkedin/datahub/blob/0116a955cbc3fe0fdba6e0effecad3c6791f8862/datahub-web/@datahub/shared/addon/components/institutional-memory/containers/tab.ts
Whitelist https://github.com/linkedin/datahub/blob/5072ef013ce3ccb116726fda38fefb7d8319fd11/datahub-web/@datahub/shared/types/configurator/configurator.d.ts
@andrewkantor andrewkantor added the bug Bug report label Sep 15, 2020
@mars-lan
Copy link
Contributor

Thank you for pointing these out, @andrewkantor. We too are working on a similar initiative internally at LinkedIn. Will try to prioritize these fixes.

@cptran777
Copy link
Contributor

Thanks for the callout @andrewkantor , we have this as a task to address internally (which would then rollout to open source as well) but haven't allocated time towards this task. Having it be called out gives a good push to prioritize more inclusive wording.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants