-
Notifications
You must be signed in to change notification settings - Fork 308
Add CI workflow to publish javascript packages #707
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ |
| @@ -1,5 +1,5 @@ | |||
| { | |||
| "name": "@pythnetwork/pyth-price-pusher", | |||
| "name": "@pythnetwork/price-pusher", | |||
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.
this hasn't been published yet so fix the name while i'm at it
| { | ||
| "name": "@pythnetwork/pyth-aptos-js", | ||
| "version": "1.0.0", | ||
| "version": "1.0.1", |
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.
nothing has changed code-wise, but i want to make sure this one and the one below get published
| types: | ||
| - created | ||
| jobs: | ||
| publish-solidity-sdk: |
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.
| publish-solidity-sdk: | |
| publish-js-package: |
I think this should work, but we'll have to try it live to find out. See the diff of README to understand how it works.
Note that this publishes the chain-specific JS sdks even though we're planning to deprecate those. It was easy to do, and this lets us start deprecating the other git repos immediately.