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

Add basics/program-derived-addresses/steel #181

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

robertohuertasm
Copy link

@robertohuertasm robertohuertasm commented Oct 21, 2024

Adds a program derived addresses example using the Steel framework containing both Bankrun tests and Rust tests.

Update to steel 2.1

@heyAyushh heyAyushh added contribute:steel Adds program example using steel framework. program-derived-addresses labels Nov 15, 2024
Copy link
Contributor

@heyAyushh heyAyushh left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Before we can merge this PR, please:

  1. Merge main branch into your PR to get the latest updates
  2. Run pnpm fix at the project root to ensure code style consistency
  3. Remove the steel program entry from the root Cargo.toml if present
  4. Run cargo fmt and cargo clippy to ensure Rust code quality
  5. Push your changes

Let me know once you've completed these steps and I'll review again. Thanks!

@robertohuertasm robertohuertasm force-pushed the roberto.huertas/feat/program-derived-addresses-steel branch from 19808bf to 28bbc75 Compare November 23, 2024 11:40
@robertohuertasm
Copy link
Author

Hi @heyAyushh, done!

@robertohuertasm robertohuertasm force-pushed the roberto.huertas/feat/program-derived-addresses-steel branch from 28bbc75 to 79496c1 Compare December 1, 2024 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribute:steel Adds program example using steel framework. program-derived-addresses
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants