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

Admin Guide linkcheck: replace or remove commented links #335

Merged
merged 1 commit into from
Jun 2, 2023

Conversation

tahliar
Copy link
Collaborator

@tahliar tahliar commented May 31, 2023

PR creator: Description

I checked the links that were commented out for the last loc drop, and mostly didn't find any good replacements, so I've removed most of them for good. I did replace one with a link to clusterlabs.

It probably makes sense to backport this change, since the old links are presumably broken in the earlier versions, too.

PR creator: Which product versions do the changes apply to?

When opening a PR, check all versions of the documentation that your PR applies to.

  • SLE-HA 15
    • 15 next (current main, no backport necessary)
    • 15 SP4
    • 15 SP3
    • 15 SP2
    • 15 SP1
  • SLE-HA 12
    • 12 SP5
    • 12 SP4

PR reviewer only: Have all backports been applied?

The doc team member merging your PR will take care of backporting to older documents.
When opening a PR, do not set the following check box.

  • all necessary backports are done

@tahliar tahliar requested a review from taroth21 May 31, 2023 06:56
@tahliar tahliar merged commit 4ba1d65 into main Jun 2, 2023
@tahliar tahliar deleted the update-commented-links branch June 2, 2023 04:27
tahliar added a commit that referenced this pull request Jun 2, 2023
tahliar added a commit that referenced this pull request Jun 2, 2023
tahliar added a commit that referenced this pull request Jun 2, 2023
tahliar added a commit that referenced this pull request Jun 2, 2023
tahliar added a commit that referenced this pull request Jun 2, 2023
tahliar added a commit that referenced this pull request Jun 2, 2023
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.

1 participant