-
Notifications
You must be signed in to change notification settings - Fork 74
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
adopt workspace protocol #2293
adopt workspace protocol #2293
Conversation
a573331
to
7c1aaf0
Compare
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.
Just one package.json
that shouldn’t participate in the workspace. I added force:integration
to the integration PR. This is all great otherwise.
7c1aaf0
to
dfa7761
Compare
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 like the integration is green. I’m giving a preëmptive approval (contingent on the one demo/package.json
) to unblock you.
Solved by @michaelfig in #2416 |
I really didn't search for the right terms when I was trying to check for prior work. Sorry for being pushy. : |
No problem! I'm happy for it to be done. I thought Lerna was the reason I dropped this back to draft but it might have been this:
Sorry I didn't find that earlier. I'm afraid you might run into it. Of course it would be great if you also just solved, |
Refs: Agoric/agoric-sdk#9209, Agoric/agoric-sdk#9288
Description
Uses the workspace: protocol to distinguish dependencies on local package with those from a registry.
Security Considerations
n/a
Scaling Considerations
n/a
Documentation Considerations
not user facing
Testing Considerations
yarn lerna version --no-push --conventional-graduate --no-git-tag-version
produces correct versionsyarn lerna exec yarn pack
. Untarred one and verified its package.json references version numbers for other monorepo packages.getting-started
which failed last time)Compatibility Considerations
n/a
Upgrade Considerations
n/a