-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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][fs-detectors] Improve vc build
monorepo support
#8742
Conversation
vc build
monorepo supportvc build
monorepo support
vc build
monorepo supportvc build
monorepo support
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.
Self-reviewed to get discussions going. This PR has become very large, I think I can pull out the @types/node
update if folks care about scope-creep.
I'm going to say that the integration test changes are necessary. The existing integration test file was simply not extensible enough to support the multitude of scenarios we need to be testing for this feature.
While working on #8742 i found some useful error checking code buried in the CLI. This PR adds a new private package `@vercel/errors` that moves those helpful utilities into its own package so it can be used throughout the monorepo.
Related Issues
Improves how
vc build
handles monorepos. In short, this introduces monorepo manager detection logic and then some helpful defaults so users don't have to manually specify abuildCommand
orinstallCommand
when linking a project within a monorepo.📋 Checklist
Tests
yarn test-unit
Code Review