-
Notifications
You must be signed in to change notification settings - Fork 420
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(cli): fixes an issue where vendor deps were not found in monorepos #6868
fix(cli): fixes an issue where vendor deps were not found in monorepos #6868
Conversation
Co-authored-by: Espen Hovlandsdal <espen@hovlandsdal.com>
* feat(cli): build user peer-deps * chore: remove shared-modules build * test(cli): verify build config and `exports` subpaths * text(cli): add depcheck config for fixture projects * docs: update comments and test name * feat: create import map w/ installed sanity version (#6784) * feat: create import map w/ installed sanity version * fix: use correct URL
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
No changes to documentation |
Component Testing Report Updated Jun 10, 2024 4:38 PM (UTC)
|
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.
oh this is perfect. i think this will work for all cases.
807001a
to
c42f659
Compare
Merged into parent branch |
Description
FIXES SDX-1390
Fixes an issue where the path for vendor package.json did not resolve correctly in monorepos. It also changes the file path to use the package.json's dirname instead of trying to resolve again from the root.
What to review
Changes makes sense
Testing
TODO, probably need a more high level testing of the build command in the perspective of the flag
Notes for release
N/A