Skip to content

fix: vendor check_docs to avoid raw GitHub URL rate limits#3704

Merged
bartlomieju merged 1 commit into
mainfrom
fix/replace-raw-github-urls
Mar 26, 2026
Merged

fix: vendor check_docs to avoid raw GitHub URL rate limits#3704
bartlomieju merged 1 commit into
mainfrom
fix/replace-raw-github-urls

Conversation

@bartlomieju
Copy link
Copy Markdown
Contributor

Summary

  • The CI check:docs task imports from raw.githubusercontent.com/denoland/std/.../_tools/check_docs.ts which intermittently fails with 429 Too Many Requests, breaking CI on unrelated PRs
  • Vendor the script locally as tools/check_docs_lib.ts (from denoland/std's _tools/check_docs.ts + inlined resolve from _tools/utils.ts) to eliminate the external dependency
  • Both @deno/doc and @std/collections were already in the import map, so no new dependencies

Test plan

  • deno task check:docs passes locally
  • deno lint and deno fmt clean

🤖 Generated with Claude Code

The CI `check:docs` task was importing from
`raw.githubusercontent.com/denoland/std/.../check_docs.ts` which
intermittently fails with 429 Too Many Requests. Vendor the script
locally to eliminate the external dependency.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bartlomieju bartlomieju merged commit 8a2b659 into main Mar 26, 2026
8 checks passed
@bartlomieju bartlomieju deleted the fix/replace-raw-github-urls branch March 26, 2026 10:05
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