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

solana: porting over contracts #366

Merged
merged 13 commits into from
Dec 17, 2024
Merged

solana: porting over contracts #366

merged 13 commits into from
Dec 17, 2024

Conversation

aalu1418
Copy link
Collaborator

@aalu1418 aalu1418 commented Dec 12, 2024

  • copy over contracts + tests
    • cleanup .gitignore
  • enable CI cached image
  • enable CI for rust + golang tests
  • enable CI for checking generated artifacts
  • reorganize tests

aalu1418 and others added 2 commits December 12, 2024 12:21
Co-authored-by: Agustina Aldasoro <agusaldasoro@users.noreply.github.com>
Co-authored-by: Tobías Lichtig <toblich@users.noreply.github.com>
Co-authored-by: Jonghyeon Park <jonghyeon.park@smartcontract.com>
Co-authored-by: PabloMansanet <pablomansanet@gmail.com>
@aalu1418 aalu1418 force-pushed the solana/init-ccip-contracts branch from 4c326b9 to e07d96f Compare December 13, 2024 16:13
@aalu1418 aalu1418 force-pushed the solana/init-ccip-contracts branch 4 times, most recently from eb740b8 to 99166ba Compare December 13, 2024 16:58
@aalu1418 aalu1418 force-pushed the solana/init-ccip-contracts branch 4 times, most recently from ae2f2f5 to c255d58 Compare December 13, 2024 18:08
@aalu1418 aalu1418 force-pushed the solana/init-ccip-contracts branch from c255d58 to cff49ff Compare December 13, 2024 18:15
@aalu1418 aalu1418 marked this pull request as ready for review December 13, 2024 22:16
@aalu1418 aalu1418 requested a review from a team as a code owner December 13, 2024 22:16
Co-authored-by: Agustina Aldasoro <agusaldasoro@users.noreply.github.com>
@@ -0,0 +1,203 @@
name: Solana
Copy link
Contributor

Choose a reason for hiding this comment

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

should we migrate the .pre-commit-config.yaml as well?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this is just the minimum set to keep developing - if you want to migrate it, you're definitely welcome to make another PR :)

agusaldasoro
agusaldasoro previously approved these changes Dec 16, 2024
Copy link
Contributor

@agusaldasoro agusaldasoro left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

Copy link

Metric solana/init-ccip-contracts main
Coverage 76.8% 76.7%

@aalu1418 aalu1418 merged commit 814a10a into main Dec 17, 2024
17 checks passed
@winder winder deleted the solana/init-ccip-contracts branch December 17, 2024 18:37
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