Skip to content

Add link checker for doc build and test#229

Merged
rapids-bot[bot] merged 14 commits intoNVIDIA:branch-25.08from
rgsl888prabhu:add_link_checker
Jul 28, 2025
Merged

Add link checker for doc build and test#229
rapids-bot[bot] merged 14 commits intoNVIDIA:branch-25.08from
rgsl888prabhu:add_link_checker

Conversation

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

Description

The link checker will go through all the links and flag any broken links in the doc

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Jul 23, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rgsl888prabhu rgsl888prabhu self-assigned this Jul 23, 2025
@rgsl888prabhu rgsl888prabhu added doc Improvements or additions to documentation non-breaking Introduces a non-breaking change labels Jul 23, 2025
@rgsl888prabhu rgsl888prabhu added this to the 25.08 milestone Jul 25, 2025
@rgsl888prabhu rgsl888prabhu marked this pull request as ready for review July 25, 2025 21:57
@rgsl888prabhu rgsl888prabhu requested review from a team as code owners July 25, 2025 21:57
Copy link
Copy Markdown
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

Looks great!

In a follow-up PR, you should fix anything that's redirecting permanently, like these:

(             faq: line   10) redirect  https://hub.docker.com/r/nvidia/ - permanently to https://hub.docker.com/u/nvidia
(    introduction: line  122) redirect  https://www.ampl.com/ - permanently to https://ampl.com/

Benefits of that:

  • faster page loads for docs
  • protection against those redirects turning into broken links in the future if the redirection is ever removed

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot rapids-bot bot merged commit 204c1d0 into NVIDIA:branch-25.08 Jul 28, 2025
281 of 286 checks passed
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
The link checker will go through all the links and flag any broken links in the doc

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Trevor McKay (https://github.com/tmckayus)
  - James Lamb (https://github.com/jameslamb)

URL: NVIDIA#229
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Improvements or additions to documentation non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants