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

banking_stage: use iproduct! macro in test #3054

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

AshwinSekar
Copy link

follow up from #2901 (comment)

@AshwinSekar
Copy link
Author

@bw-solana the other follow up #2901 (comment) (making the fn private) is now obsolete due to #3049.

Also since this change only updates testing code, I don't think we need to backport it.

@AshwinSekar AshwinSekar requested a review from bw-solana October 2, 2024 15:25
Copy link

@bw-solana bw-solana left a comment

Choose a reason for hiding this comment

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

LGTM

@AshwinSekar AshwinSekar merged commit 14fe368 into anza-xyz:master Oct 2, 2024
40 checks passed
@AshwinSekar AshwinSekar deleted the iproduct branch October 2, 2024 15:40
ray-kast pushed a commit to abklabs/agave that referenced this pull request Nov 27, 2024
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.

2 participants