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

Support --upgradeable-program in anchor test #2642

Merged

Conversation

nickguo
Copy link
Contributor

@nickguo nickguo commented Sep 29, 2023

please advise on how to best test this change

@vercel
Copy link

vercel bot commented Sep 29, 2023

@nickguo is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

cli/src/lib.rs Outdated Show resolved Hide resolved
@nickguo nickguo force-pushed the anchor_test_upgradeable_program_flag branch 3 times, most recently from 265ae11 to 2d00b48 Compare September 29, 2023 22:43
cli/src/config.rs Outdated Show resolved Hide resolved
cli/src/lib.rs Show resolved Hide resolved
@nickguo nickguo force-pushed the anchor_test_upgradeable_program_flag branch from 2d00b48 to 65032e2 Compare October 2, 2023 05:17
Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

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

LGTM other than a couple of small issues I've mentioned.

Could you add an entry to the CHANGELOG before we merge?

docs/src/pages/docs/manifest.md Outdated Show resolved Hide resolved
docs/src/pages/docs/manifest.md Outdated Show resolved Hide resolved
@nickguo nickguo force-pushed the anchor_test_upgradeable_program_flag branch 2 times, most recently from 2ff4e67 to e42da26 Compare October 2, 2023 19:57
Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

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

There is a conflict in the CHANGELOG file that blocks the merge. Could you rebase to master?

CHANGELOG.md Show resolved Hide resolved
@nickguo nickguo force-pushed the anchor_test_upgradeable_program_flag branch from e42da26 to c2c8ecd Compare October 2, 2023 22:45
cli/src/config.rs Outdated Show resolved Hide resolved
@nickguo nickguo force-pushed the anchor_test_upgradeable_program_flag branch from c2c8ecd to f076e7d Compare October 3, 2023 13:12
@acheroncrypto
Copy link
Collaborator

Thank you for the contribution and addressing the feedback quickly!

@acheroncrypto acheroncrypto merged commit 5602244 into coral-xyz:master Oct 3, 2023
46 of 47 checks passed
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.

2 participants