generated from linux-system-roles/template
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Eliminate non-inclusive language #59
Comments
I have one more question. I'd like to apply this similar changes to |
As #64 is merged, closing this issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello, we have a project to eliminate non-inclusive language from the linux-system-roles.
Running a utility woke (now it's supported in tox. please install the latest tox-lsr and run tox -e woke), two non-inclusive words are reported -
sanity
andmaster
.The difficulty is
ControlMaster
is a configuration keyword forssh
. So, unless it's renamed inssh
, there is no way to avoid it in thessh role
. To skip the checking, there are two ways - adding wokeignore:rule=master to eachControlMaster
appearing in the source code, or skipping them as warning by applying get-woke/woke#252. Which way is preferable?The text was updated successfully, but these errors were encountered: