Skip to content

docs: note that deno task ** recurses into node_modules#3187

Open
bartlomieju wants to merge 1 commit into
mainfrom
docs/deno-task-globstar-node-modules
Open

docs: note that deno task ** recurses into node_modules#3187
bartlomieju wants to merge 1 commit into
mainfrom
docs/deno-task-globstar-node-modules

Conversation

@bartlomieju

Copy link
Copy Markdown
Member

Adds a caution note to the deno task glob expansion docs clarifying that,
because globstar is enabled by default, ** recurses into all descendant
directories including node_modules. This surprises users coming from shells
where ** is not recursive by default: a task like deno check **/*.ts can
expand to far more files than the same command typed in the terminal, leading to
errors like Argument list too long or type-checking unintended dependency
files.

The note points to running deno check / deno fmt without glob arguments plus
the exclude config (which excludes node_modules by default), or disabling
recursion per-task with shopt -u globstar.

Prompted by denoland/deno#31910.

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