Skip to content

add hello solana asm example#538

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

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

Conversation

@Perelyn-sama
Copy link
Copy Markdown
Collaborator

No description provided.

@dev-jodee dev-jodee self-requested a review March 24, 2026 12:44
Copy link
Copy Markdown
Collaborator

@dev-jodee dev-jodee left a comment

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
Copy link
Copy Markdown
Collaborator

@dev-jodee dev-jodee left a comment

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
Collaborator 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-developers: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