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

SDK: Restructure project workspace #432

Merged
merged 14 commits into from
Jun 14, 2024
Merged

SDK: Restructure project workspace #432

merged 14 commits into from
Jun 14, 2024

Conversation

barnjamin
Copy link
Contributor

@barnjamin barnjamin commented Apr 25, 2024

Restructure package workspace to share TS across the entire repo and keep lib/sdk code alongside contract source.

Moves /solana/ts/sdk to /solana/ts/lib
Moves /sdk/solana to /solana/ts/sdk

Checks in single copy of each version of idl to /solana/ts/idl/{json,ts}

Update /solana/ts/lib and existing anchor test to rely only on SDKv2

Update paths in Dockerfiles, Makefiles, imports for CI tests

Deduplicate Solana ts code (#442)

@barnjamin barnjamin changed the title SDK: Solana updates to account for Token22 SDK: Restructure project workspace, add Token22 support Apr 27, 2024
@barnjamin barnjamin marked this pull request as ready for review April 29, 2024 14:05
@barnjamin barnjamin changed the title SDK: Restructure project workspace, add Token22 support SDK: Restructure project workspace Apr 29, 2024
@kcsongor kcsongor force-pushed the solana/token22 branch 4 times, most recently from 6b32e93 to 1ac742d Compare April 29, 2024 16:36
@kcsongor kcsongor force-pushed the solana/token22-ci branch from c6541a0 to e943407 Compare April 29, 2024 16:57
@barnjamin barnjamin mentioned this pull request Apr 30, 2024
6 tasks
@barnjamin barnjamin force-pushed the solana/token22-ci branch from 0e023e8 to 7f112c9 Compare May 1, 2024 14:58
@kcsongor kcsongor force-pushed the solana/token22 branch 4 times, most recently from c8bc683 to 4b564c0 Compare May 2, 2024 16:37
@barnjamin barnjamin force-pushed the solana/token22-ci branch from 7f112c9 to f00c432 Compare May 2, 2024 16:47
@barnjamin barnjamin force-pushed the solana/token22-ci branch from 1422805 to 65e4e05 Compare June 6, 2024 15:32
@barnjamin barnjamin force-pushed the solana/token22-ci branch 2 times, most recently from 0840cb9 to 5211f1f Compare June 7, 2024 20:04
Base automatically changed from solana/token22 to main June 12, 2024 18:20
@barnjamin barnjamin force-pushed the solana/token22-ci branch from 5211f1f to f8acdab Compare June 13, 2024 12:23
Copy link
Contributor

@kcsongor kcsongor left a comment

Choose a reason for hiding this comment

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

I've been using this branch for a while and everything has worked as expected.

@barnjamin barnjamin removed the on hold label Jun 13, 2024
Copy link
Collaborator

@nonergodic nonergodic left a comment

Choose a reason for hiding this comment

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

only gave it a quick-ish look-over the second time around but it's clearly a great improvement already!

@barnjamin barnjamin merged commit 5d3a0d5 into main Jun 14, 2024
6 checks passed
@barnjamin barnjamin deleted the solana/token22-ci branch June 14, 2024 10:36
bruce-riley pushed a commit to wormholelabs-xyz/native-token-transfers that referenced this pull request Sep 12, 2024
* SDK: Migrate Solana TS code to Solana dir (wormhole-foundation#433)

* bump to stable version of sdk

* Move generated IDL to ts directory (wormhole-foundation#437)

* Move generated IDL to ts directory

* update github action with new idl path

* Update to use the ts exported IDL instead of the json

* rebase and regenerate bindings

* remove generics from generated idl

* add generic stripper to makefile, use ts IDL for binding loader

* update bindings to work for both versions (wormhole-foundation#439)

* remove dupe quoter implementation, pass args to createRequestRelayInstruction

* Partial CR fixes

* Update makefile with suggestions from CR

* skip anchor build on test

* SDK: Solana TS deduplication (wormhole-foundation#442)

* set peer dep for ntt route

* update version

* prev commit for tilt to test

* restore tiltfile git checkout to main

* Add 1.1.0 evm bindings (wormhole-foundation#457)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants