Fix dead tutorial links flagged by link-rot audit - #5264
Open
jeremyrose-viam wants to merge 1 commit into
Open
Conversation
✅ Deploy Preview for viam-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
viambot
added
the
safe to build
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #5196 (recurring
run-htmltest-externalCI failure on the Amazon 24Vpower supply link in
claw-game.md), plus three more dead links found whilelooking into that same tutorial and running a one-off link audit:
docs/tutorials/projects/claw-game.md: replaced the dead Amazon 24V powersupply link and the dead eBay arcade-claw link (auction had ended; not
caught by CI because
ebay.comis in.htmltest.yml'sIgnoreURLs).docs/tutorials/projects/integrating-viam-with-openai.md: replaced a deadElevenLabs auth-docs link with the ElevenLabs dashboard API-keys page
(added
elevenlabs.io/appto.htmltest.yml's ignore list, since it's anauthenticated client-rendered SPA that a plain HTTP check can't verify
either way — same reasoning as the existing
app.viam.comentry). Alsoswapped a dead Fandom wiki link for a Wikipedia one in the same file.
All four pre-PR checks (prettier, markdownlint, vale,
make build-prod)pass clean.
Context
While fixing the CI-flagged Amazon link, noticed the eBay link on the same
page was also dead but invisible to CI (ignore-listed domain), which raised
a bigger question about the amount of unmaintained tutorial content in this
repo. Wrote a quick audit script to check every external link under
docs/tutorials/(not just what htmltest is willing to check) against.htmltest.yml's ignore list — it's on a separate branch,add-tutorial-link-audit-script,not included in this PR since we're not adding it to the repo at this time.
Running it against all tutorials turned up the 3 additional dead links fixed
here; everything else checked out clean.