Skip to content

Migrate from solana-pubkey to solana-address#90

Merged
lorisleiva merged 4 commits intocodama-idl:mainfrom
grod220:solana-address
Mar 9, 2026
Merged

Migrate from solana-pubkey to solana-address#90
lorisleiva merged 4 commits intocodama-idl:mainfrom
grod220:solana-address

Conversation

@grod220
Copy link
Copy Markdown
Contributor

@grod220 grod220 commented Mar 5, 2026

  • Makes replacement across templates, generated code, and dependency defs
  • Upgrade borsh from v0.10 to v1.0 and solana crate versions (required by solana-address)
  • Also deleted the stale cargo.lock files (given workspace deps, everything defers to root lock file)

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 6, 2026

🦋 Changeset detected

Latest commit: ed3633a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@codama/renderers-rust Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@grod220 grod220 marked this pull request as ready for review March 6, 2026 22:00
@grod220
Copy link
Copy Markdown
Contributor Author

grod220 commented Mar 6, 2026

@lorisleiva request for review 🙏

@lorisleiva lorisleiva self-requested a review March 7, 2026 10:59
Copy link
Copy Markdown
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

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

Thanks! I agree this warrants a new major version. Just a small lint issue raised by CI. IIRC you can run pnpm lint:fix to auto-fix these.

@lorisleiva lorisleiva merged commit f214970 into codama-idl:main Mar 9, 2026
4 checks passed
@github-actions github-actions bot mentioned this pull request Mar 9, 2026
@grod220 grod220 deleted the solana-address branch March 9, 2026 12:38
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