Skip to content

tools: limit checkout for lint-js-and-md - #66963

Draft
inoway46 wants to merge 1 commit into
nodejs:mainfrom
inoway46:lint-js-md-sparse-checkout
Draft

inoway46 wants to merge 1 commit into
nodejs:mainfrom
inoway46:lint-js-md-sparse-checkout

Conversation

@inoway46

Copy link
Copy Markdown
Contributor

Exclude deps/openssl/ and V8 files outside deps/v8/include/ from the lint-js-and-md checkout. Keep the V8 headers for Markdown link validation, and keep all other paths included by default.

This removes about 429 MiB (62%) of tracked file size, uncompressed. JS/MD linting and man page verification passed locally with the sparse checkout.

The ubuntu-slim comparison, with 128 trials per variant and caches disabled, is still in progress.

Refs: #64972

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Oct 23, 2026
Exclude OpenSSL and V8 files outside deps/v8/include from the JS/MD
lint checkout. Keep the V8 headers for Markdown link validation and
include all other paths by default.

This reduces the tracked checkout size by about 429 MiB without
changing the runner or lint targets.

Signed-off-by: inoway46 <[email protected]>
Assisted-by: Codex
@inoway46
inoway46 force-pushed the lint-js-md-sparse-checkout branch from 56a0597 to 88231b7 Compare October 23, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants