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

ci: Test anchor-spl during downstream build #34368

Merged
merged 4 commits into from
Dec 9, 2023

Conversation

joncinque
Copy link
Contributor

Problem

According to the Anchor team, we should also make sure that anchor-spl works during our downstream checks.

Summary of Changes

Add a step to test anchor-spl with all features enabled.

@acheroncrypto I can't add you as a reviewer, but can you double-check that this is correct?

Note: This is blocked by coral-xyz/anchor#2722, so keeping it in draft

Fixes #

@joncinque joncinque requested a review from yihau December 8, 2023 12:31
Co-authored-by: acheron <98934430+acheroncrypto@users.noreply.github.com>
@joncinque
Copy link
Contributor Author

@acheroncrypto btw, since we checkout a version tag for the downstream build, we'll keep getting the error on the build since 0.29.0 has the issue. do you plan to tag 0.29.1 or should I wait until 0.30.0 to add the --all-features test?

@acheroncrypto
Copy link
Contributor

Can we just enable the features we want in order to make this work now without requiring a new release? rust-lang/cargo#11467 would help a lot here but for now, we can do:

--features dex metadata stake

These are the additional features we'd want to test since token features are enabled by default.

@joncinque
Copy link
Contributor Author

Ok great, done!

@joncinque joncinque marked this pull request as ready for review December 8, 2023 20:06
Copy link
Member

@yihau yihau left a comment

Choose a reason for hiding this comment

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

🔥

@joncinque joncinque merged commit a1d2d6f into solana-labs:master Dec 9, 2023
16 checks passed
@joncinque joncinque deleted the anchorspl branch December 9, 2023 00:50
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