Skip to content

Add check for non-inclusive language - #90

Merged
martinpitt merged 2 commits into
linux-system-roles:mainfrom
nhosoi:woke-action
Dec 20, 2022
Merged

martinpitt merged 2 commits into
linux-system-roles:mainfrom
nhosoi:woke-action

Conversation

@nhosoi

@nhosoi nhosoi commented Dec 14, 2022 •

Copy link
Copy Markdown
Contributor

Add a check for usage of terms and language that is considered non-inclusive. We are using the woke tool for this with a wordlist that can be found at https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/config_files/woke.yml

CHANGELOG.md - cleanup non-inclusive words.

Comment thread .github/workflows/woke.yml Outdated
@nhosoi
nhosoi force-pushed the woke-action branch 2 times, most recently from c04e9f0 to 56dbe20 Compare December 14, 2022 18:29
@nhosoi nhosoi changed the title [WIP] Add github action woke.yml (github.com/get-woke/woke-action) [WIP] Add check for non-inclusive language Dec 14, 2022
Add a check for usage of terms and language that is considered
non-inclusive. We are using the woke tool for this with a wordlist
that can be found at
https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/config_files/woke.yml

Signed-off-by: Noriko Hosoi <nhosoi@redhat.com>
@nhosoi nhosoi changed the title [WIP] Add check for non-inclusive language Add check for non-inclusive language Dec 16, 2022
@nhosoi

nhosoi commented Dec 16, 2022

Copy link
Copy Markdown
Contributor Author

[citest]

@richm

richm commented Dec 16, 2022

Copy link
Copy Markdown
Contributor

[ citest ]

I don't think you need to CI test this PR?

@nhosoi

nhosoi commented Dec 16, 2022

Copy link
Copy Markdown
Contributor Author

[ citest ]

I don't think you need to CI test this PR?

Yeah, I wanted to make sure I really did not break anything... But I cannot do it with the files I modified in this PR... :p

@richm
richm requested a review from martinpitt December 16, 2022 02:59
@martinpitt

Copy link
Copy Markdown
Contributor

Do you know why the Fedora-37/ansible-2.13/(citool) test is shown as failure? There's no error message/failing test there.

Comment thread CHANGELOG.md
@nhosoi

nhosoi commented Dec 19, 2022

Copy link
Copy Markdown
Contributor Author

@martinpitt, could you please review this pr one more time? If it looks good, may I ask to merge it? Thanks in advance.

@martinpitt
martinpitt merged commit 16fcfa5 into linux-system-roles:main Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants