Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix spurious detection of yarn #243250

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix spurious detection of yarn #243250

wants to merge 2 commits into from

Conversation

rotu
Copy link

@rotu rotu commented Mar 11, 2025

Fixes #170101
Partial #193001

Previously, vscode could use find-yarn-workspace-root to identify that a package is using yarn as its package manager by looking for a package.json with the workspaces key. That is incorrect - npm and bun also use this key in their package manifest.

Now when running scripts from package.json in a multi-workspace package, it should default to npm. It should still use yarn if a yarn.lock file is detected.

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.

npm.packageManager=auto incorrectly assumes yarn as package manager for running tasks
2 participants