CI: Skip link-check on external URIs - #3966
Merged
kevinjqliu merged 2 commits intoSep 14, 2026
Merged
Conversation
rambleraptor
force-pushed
the
markdown-link-check-retry-429
branch
from
September 14, 2026 20:40
e7b51b6 to
814c3b6
Compare
rambleraptor
changed the title
Collaborator
Author
|
@kevinjqliu I'm seeing many of our PRs failing on this check. Ray seems to be very consistently throwing 429s at GitHub actions only (it's working fine on my local machine) |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
rambleraptor
force-pushed
the
markdown-link-check-retry-429
branch
from
September 14, 2026 21:17
814c3b6 to
c28130d
Compare
External docs hosts rate-limit the markdown link checker, so note that external URLs in docs need a markdown-link-check-disable-next-line comment. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V2AEta8dDFjjxotEvYnb3D
rambleraptor
force-pushed
the
markdown-link-check-retry-429
branch
from
September 14, 2026 21:19
c28130d to
2ec6397
Compare
rambleraptor
changed the title
Contributor
|
lets see if this works. otherwise we can be more aggressive with configuring the markdown link check |
Collaborator
Author
|
This should get us fixed in the interim. It looks like most of this afternoon's PRs are stuck on the same failed link. |
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.
Rationale for this change
The Ray docs are consistently throwing 429s at GitHub Actions, which is causing our Markdown Link checker to consistently fail.
At @kevinjqliu's suggestion, I'm adding disable lines to external URIs in the docs. Additionally, I've added in some info on AGENTS.md so that future external URIs (written by bots) will hopefully get these in the future.
Are these changes tested?
CI should remain passing.
Are there any user-facing changes?