Skip to content

add hello solana asm example#538

Merged
Perelyn-sama merged 5 commits into
solana-foundation:mainfrom
Perelyn-sama:asm/hello-world
Apr 13, 2026
Merged

add hello solana asm example#538
Perelyn-sama merged 5 commits into
solana-foundation:mainfrom
Perelyn-sama:asm/hello-world

Conversation

@Perelyn-sama

Copy link
Copy Markdown
Contributor

No description provided.

@dev-jodee
dev-jodee self-requested a review March 24, 2026 12:44

@dev-jodee dev-jodee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Couple of small things to change :) thx for the work

Comment thread .gitignore
Comment thread basics/hello-solana/asm/src/lib.rs
Comment thread Cargo.toml Outdated
Comment thread basics/hello-solana/asm/cicd.sh Outdated
@Perelyn-sama
Perelyn-sama requested a review from dev-jodee March 24, 2026 13:39

@dev-jodee dev-jodee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

re on the solana dependency added -> Yes, but that dependency isn't used anywhere in your code itself

@Perelyn-sama

Copy link
Copy Markdown
Contributor Author

re on the solana dependency added -> Yes, but that dependency isn't used anywhere in your code itself

fair, removed.

Also, thanks for the reviews. feels good to be making progress :)

@dev-jodee

Copy link
Copy Markdown
Collaborator

@Perelyn-sama of course ! and can you fix the conflict? after that I think we'll be good to merge

@dev-jodee
dev-jodee self-requested a review April 13, 2026 14:43
@Perelyn-sama
Perelyn-sama merged commit 8d1f1a7 into solana-foundation:main Apr 13, 2026
15 checks passed
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