Skip to content

Conversation

@jayantk
Copy link
Contributor

@jayantk jayantk commented Feb 17, 2023

This PR moves the contents of https://github.com/pyth-network/pyth-js into this repository. I did the following:

  • Moved most of target_chains/ethereum/ to target_chains/ethereum/contracts to match the organization of aptos etc. exception is the examples folder which stayed in ethereum/examples
  • Copied each javascript sdk from pyth-js to chain_name/js_sdk
  • Copied the price pusher to the top-level price_pusher directory. (I think we will probably generalize this to support more chains, so top level made sense)

I have tried to keep this to moving/copying files, so I haven't changed any dependencies, package.jsons, or docs beyond what was necessary to get ci to pass. I will follow this up with a PR fixing all of that, plus the build & release process for the js libraries.

@vercel
Copy link

vercel bot commented Feb 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated
example-oracle-amm ⬜️ Ignored (Inspect) Feb 21, 2023 at 10:36PM (UTC)
xc-admin-frontend ⬜️ Ignored (Inspect) Feb 21, 2023 at 10:36PM (UTC)

@jayantk jayantk changed the title [wip] Move JS sdks into this repo [sdk] Move JS sdks into this repo Feb 21, 2023
Copy link
Collaborator

@ali-behjati ali-behjati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much. I ran tilt locally to make sure it works and it worked.

I think the next steps are adding the new packages to lerna, having publish action for them and image push action for the price pusher.

@jayantk
Copy link
Contributor Author

jayantk commented Feb 22, 2023

tested with tilt locally.

@jayantk jayantk merged commit bbe10ba into main Feb 22, 2023
@jayantk jayantk deleted the move_sdks branch February 22, 2023 21:44
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