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

Remove CLP module from documentation #3127

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Remove CLP module from documentation #3127

merged 1 commit into from
Jul 16, 2024

Conversation

AkshayGadhaveRH
Copy link
Contributor

With the launch of docs.redhat.com, the conscious language module is no longer required as the inclusivity statement is a part of the footnotes. E.g.: https://docs.redhat.com/en/documentation/red_hat_satellite/6.15/html/installing_satellite_server_in_a_connected_network_environment/index Removing the CLP module for all releases and from all guides.

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.11/Katello 4.13
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9)
  • 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)
  • We do not accept PRs for Foreman older than 3.5.

Remove CLP module from documentation
83ad8b6 
With the launch of docs.redhat.com, the conscious language module is no longer required as the inclusivity statement is a part of the footnotes.
E.g.: https://docs.redhat.com/en/documentation/red_hat_satellite/6.15/html/installing_satellite_server_in_a_connected_network_environment/index
Removing the CLP module for all releases and from all guides.
@github-actions GitHub Actions
Copy link

The PR preview for 83ad8b6 is available at theforeman-foreman-documentation-preview-pr-3127.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

Lennonka
Lennonka approved these changes Jul 15, 2024
View reviewed changes
Copy link
Contributor

@Lennonka Lennonka left a comment •
edited

Choose a reason for hiding this comment

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

LGTM, thank you!
I'll merge it tomorrow.

maximiliankolb
maximiliankolb approved these changes Jul 16, 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 2aee327 into theforeman:master Jul 16, 2024
7 of 8 checks passed
@Lennonka
Copy link
Contributor

Lennonka commented Jul 16, 2024 •
edited

Cherry-picked:

The module isn't present in earlier branches. Nothing to cherry pick in 3.8--3.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maximiliankolb maximiliankolb maximiliankolb approved these changes

@Lennonka Lennonka Lennonka 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.

None yet
3 participants
@AkshayGadhaveRH @Lennonka @maximiliankolb