-
Notifications
You must be signed in to change notification settings - Fork 68.8k
internal: Update stale link checker documentation #45508
Copy link
Copy link
Closed
Labels
contentThis issue or pull request belongs to the Docs Content teamThis issue or pull request belongs to the Docs Content teamfix-internallyTriggers a workflow to copy the issue internally and close the current issueTriggers a workflow to copy the issue internally and close the current issuetriageDo not begin working on this issue until triaged by the teamDo not begin working on this issue until triaged by the team
Description
Activity
Metadata
Metadata
Assignees
Labels
contentThis issue or pull request belongs to the Docs Content teamThis issue or pull request belongs to the Docs Content teamfix-internallyTriggers a workflow to copy the issue internally and close the current issueTriggers a workflow to copy the issue internally and close the current issuetriageDo not begin working on this issue until triaged by the teamDo not begin working on this issue until triaged by the team
To contributors: this change can only be done by internal Github maintainers. Any PR is automatically closed.
Code of Conduct
What article on docs.github.com is affected?
While working on #45507, I noticed that
src/links/README.mdstill describes the previous rendered-content link checker.The old
rendered-content-link-checker.tsandrendered-content-link-checker-cli.tsscripts were removed when the link-checking system was rebuilt. However, the README still:rendered-content-link-checker.tsscript.rendered-content-link-checker-clinpm command.--leveland--check-external-links.LinkPreviewPopover.tsxwithout its currentcomponents/path.The current link-checking commands are:
npm run check-links-prnpm run check-links-internalnpm run check-links-externalWhat changes are you suggesting?
Update the README’s architecture, usage examples, supported options, component path, and performance notes to reflect the current link-checking system.
This should be a documentation-only change. I closed #45507 so the gap could be discussed and confirmed before proposing another update.
I may be missing context about how this internal README is intended to be maintained, so I’d appreciate confirmation that bringing it in line with the current scripts would be useful.
Additional information
No response