Skip to content

fix: quote glob in package script#1010

Open
clemyan wants to merge 2 commits into
vuejs:mainfrom
clemyan:quote-glob
Open

fix: quote glob in package script#1010
clemyan wants to merge 2 commits into
vuejs:mainfrom
clemyan:quote-glob

Conversation

@clemyan
Copy link
Copy Markdown

@clemyan clemyan commented May 11, 2026

Description

When using oxlint, the lint script in the generated package.json contains a * that may be expanded by the shell and can lead to unexpected behavior

This PR wraps the arg in quotes to stop shells from expanding it as a glob

This should fix #951, but I can't confirm right now

When using oxlint, the `lint` script in the generated `package.json`
contains a `*` that may be expanded by the shell and can lead to
expected behavior

This commit adds wraps the arg in quotes to stop shells from expanding
it as a glob
Comment thread template/linting/oxlint/package.json Outdated
Co-authored-by: skirtle <65301168+skirtles-code@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@skirtles-code skirtles-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

This fix is also needed for Yarn, not just bun.

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.

Linter is not working when used with bun on Windows

2 participants