-
Notifications
You must be signed in to change notification settings - Fork 321
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/transfer-sol/steel #201
Conversation
There was a problem hiding this 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:
- Merge main branch into your PR to get the latest updates
- Run
pnpm fix
at the project root to ensure code style consistency - Remove the steel program entry from the root Cargo.toml if present
- Run
cargo fmt
andcargo clippy
to ensure Rust code quality - Push your changes
Let me know once you've completed these steps and I'll review again. Thanks!
I followed the instructions you told me, but I see a change in the contribution guide, so should I change the script according to the new instructions or leave it as is, but the job check I see something wrong even with hello-solana/steel which I merged from the main branch when building also fails @heyAyushh. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.