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.

Sign up for GitHub

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

Jump to bottom

Fix inclusive language issues #2790

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Fix inclusive language issues #2790

merged 2 commits into from
Feb 19, 2024

Conversation

apinnick
Copy link
Contributor

@apinnick apinnick commented Feb 15, 2024 •
edited
Loading

  • Replace blacklist and whitelist according to inclusive language guidelines.

  • Only generic text is changed.

  • Command blocks are unchanged.

  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.9/Katello 4.11 (planned Satellite 6.15)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6/6.7)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only; orcharhino 6.4/6.5 on EL8 only)
  • We do not accept PRs for Foreman older than 3.3.

@apinnick
Inclusive language
e2bd4b8 
* Replace blacklist and whitelist according to inclusive language guidelines.
* Only generic text is changed.
* Command blocks are unchanged.
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 15, 2024 •
edited
Loading

The PR preview for b86c9be is available at theforeman-foreman-documentation-preview-pr-2790.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

@apinnick apinnick changed the title Inclusive language Fix inclusive language issues Feb 15, 2024
maximiliankolb
maximiliankolb requested changes Feb 19, 2024
View reviewed changes
Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two threads remain open; rest LGTM

guides/common/modules/ref_skipping-steps-when-performing-backups.adoc Outdated Show resolved Hide resolved
guides/common/modules/proc_configuring-http-proxy-to-connect-to-cdn.adoc Outdated Show resolved Hide resolved
@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author Needs re-review and removed Waiting on contributor Requires an action from the author labels Feb 19, 2024
@apinnick
Copy link
Contributor Author

Looks like I forgot to push the commit. Done

maximiliankolb
maximiliankolb approved these changes Feb 19, 2024
View reviewed changes
Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Lennonka Lennonka merged commit 3b0b91f into theforeman:master Feb 19, 2024
8 checks passed
Lennonka pushed a commit that referenced this pull request Feb 19, 2024
@apinnick @Lennonka
Inclusive language (#2790)
0642a5c 
* Replace blacklist and whitelist according to inclusive language guidelines.
* Only generic text is changed.
* Command blocks are unchanged.
@Lennonka
Copy link
Contributor

Cherry-picked:

@apinnick apinnick deleted the inclusive-language branch March 31, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ekohl ekohl ekohl left review comments

@maximiliankolb maximiliankolb maximiliankolb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@apinnick @Lennonka @ekohl @maximiliankolb