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
[cli] Use getNodeBinPaths()
in vc dev
#10225
Conversation
This allows for the "dev command" of a Project to work better in monorepos, where the dev server might live up the node_modules hierarchy within the repo.
🦋 Changeset detectedLatest commit: f2351a8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just needs tests 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR'd and looks good! Did not test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think tests need updated here since this seems like you are just replacing a previous api. LGTM 🚀
# Releases ## vercel@31.1.0 ### Minor Changes - Add 'Environment' column to 'vc list' with new '--environment' filter and pipe URLs to stdout ([#10239](#10239)) ### Patch Changes - Update `proxy-agent` to v6.3.0 ([#10226](#10226)) - Use `getNodeBinPaths()` in `vc dev` ([#10225](#10225)) - Updated dependencies \[[`b1c14cde0`](b1c14cd), [`ce4633fe4`](ce4633f)]: - @vercel/next@3.9.1 - @vercel/static-build@1.3.42 ## @vercel/frameworks@1.5.0 ### Minor Changes - Add `ignorePackageJsonScript` configuration for Framework command settings to ignore the `package.json` script. ([#10228](#10228)) Enable this mode for Storybook's `buildCommand`, since it should not invoke the "build" script, which is most likely designated for the frontend app build. ## @vercel/fs-detectors@4.1.1 ### Patch Changes - Updated dependencies \[[`ce4633fe4`](ce4633f)]: - @vercel/frameworks@1.5.0 ## @vercel/next@3.9.1 ### Patch Changes - Fix pages and app router i18n handling ([#10243](#10243)) ## @vercel/static-build@1.3.42 ### Patch Changes - Add `ignorePackageJsonScript` configuration for Framework command settings to ignore the `package.json` script. ([#10228](#10228)) Enable this mode for Storybook's `buildCommand`, since it should not invoke the "build" script, which is most likely designated for the frontend app build.
This allows for the "dev command" of a Project to work better in monorepos, where the dev server might live up the node_modules hierarchy within the repo.