Skip to content

fix: replace non-portable find -delete with del-cli in clean script#2306

Merged
kamilmysliwiec merged 2 commits intov12.0.0from
copilot/sub-pr-2302-again
Mar 11, 2026
Merged

fix: replace non-portable find -delete with del-cli in clean script#2306
kamilmysliwiec merged 2 commits intov12.0.0from
copilot/sub-pr-2302-again

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 11, 2026

The clean script used find ... -delete, a Unix-only command that breaks npm test on Windows since test depends on clean.

Changes

  • package.json: Replaced find ... -delete with del-cli, which supports glob negation patterns and runs cross-platform
"clean": "del 'src/**/*.{js,d.ts,js.map,d.ts.map}' '!src/**/files/**' '!src/**/workspace/**' '!src/**/*.schema.d.ts'"

The exclusions for files/, workspace/, and *.schema.d.ts are preserved exactly as before.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: kamilmysliwiec <23244943+kamilmysliwiec@users.noreply.github.com>
Copilot AI changed the title [WIP] Update PR to address feedback on v12.0.0 changes fix: replace non-portable find -delete with del-cli in clean script Mar 11, 2026
@kamilmysliwiec kamilmysliwiec marked this pull request as ready for review March 11, 2026 15:04
@kamilmysliwiec kamilmysliwiec merged commit 91c5c13 into v12.0.0 Mar 11, 2026
1 check passed
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.

2 participants