Skip to content

Conversation

0xCipherCoder
Copy link
Contributor

  • Updated anchor code as per guidelines and updated version
  • Added an account data example using the Steel framework using Bankrun tests.

@heyAyushh heyAyushh added contribute:steel Adds program example using steel framework. account-data basics/account-data labels Nov 15, 2024
@heyAyushh
Copy link
Member

@0xCipherCoder #198, Merge main to your branch again.

Copy link
Member

@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.

Just the CI fixes

Cargo.toml Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Remove this mention

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What is required to remove? @heyAyushh

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I removed steel from the cargo.toml file

@0xCipherCoder
Copy link
Contributor Author

@0xCipherCoder #198, Merge main to your branch again.

Thanks for the review! @heyAyushh I have merged the main branch and removed steel from cargo.toml file

@0xCipherCoder
Copy link
Contributor Author

@heyAyushh I have fixed clippy and rustfmt related formatting and style issues. Please review it once.

@0xCipherCoder
Copy link
Contributor Author

@heyAyushh I have fixed clippy and rustfmt related formatting and style issues. Please review it once.

@heyAyushh I have fixed clippy issues as well.

@heyAyushh heyAyushh merged commit 865b716 into solana-developers:main Jan 2, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
account-data basics/account-data contribute:steel Adds program example using steel framework.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants