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

adopt workspace protocol #2293

Closed
wants to merge 1 commit into from
Closed

adopt workspace protocol #2293

wants to merge 1 commit into from

Conversation

turadg
Copy link
Member

@turadg turadg commented May 20, 2024

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 versions
  • Packed all the packages with yarn lerna exec yarn pack. Untarred one and verified its package.json references version numbers for other monorepo packages.
  • Agoric integration PR: Endo integration scratch PR Agoric/agoric-sdk#9385 (it should pass getting-started which failed last time)

Compatibility Considerations

n/a

Upgrade Considerations

n/a

@turadg turadg force-pushed the ta/workspace-protocol branch 2 times, most recently from a573331 to 7c1aaf0 Compare May 21, 2024 18:26
@turadg turadg requested a review from kriskowal May 21, 2024 18:42
@turadg turadg marked this pull request as ready for review May 21, 2024 18:42
Copy link
Member

@kriskowal kriskowal left a 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.

packages/compartment-mapper/demo/policy/package.json Outdated Show resolved Hide resolved
@turadg turadg force-pushed the ta/workspace-protocol branch from 7c1aaf0 to dfa7761 Compare May 21, 2024 19:47
Copy link
Member

@kriskowal kriskowal left a 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.

@turadg turadg enabled auto-merge May 21, 2024 19:48
@turadg turadg disabled auto-merge May 21, 2024 19:49
@turadg turadg marked this pull request as draft May 21, 2024 21:00
@turadg
Copy link
Member Author

turadg commented Aug 19, 2024

Solved by @michaelfig in #2416

@turadg turadg closed this Aug 19, 2024
@turadg turadg deleted the ta/workspace-protocol branch August 19, 2024 16:54
@michaelfig
Copy link
Member

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. :

@turadg
Copy link
Member Author

turadg commented Aug 19, 2024

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:

Agoric integration PR: Agoric/agoric-sdk#9385 (it should pass getting-started which Agoric/agoric-sdk#9288)

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,

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.

3 participants