Skip to content

Conversation

@gavriliumircea
Copy link
Contributor

@gavriliumircea gavriliumircea commented Nov 6, 2025

This pr replaces #61. This is need because of a merge order mistake that I did.

@coderabbitai
Copy link

coderabbitai bot commented Nov 6, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request introduces new enum and type definitions to the Solana module. A new Protocol enum is added with Airdrops and Lockup values, alongside a Version namespace containing protocol-specific version enums. Corresponding TypeScript types are defined as unions combining template-literal types with their enum counterparts. The changes include a new Version type that combines Airdrops and Lockup versions. Public re-exports are added to src/solana/index.ts to expose the enums and types modules, expanding the module's public API surface.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/add-solana-contract-versions

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d6ab2ed and b948242.

📒 Files selected for processing (3)
  • src/solana/enums.ts (1 hunks)
  • src/solana/index.ts (1 hunks)
  • src/solana/types.ts (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor Author

gavriliumircea commented Nov 6, 2025

@gavriliumircea gavriliumircea marked this pull request as ready for review November 6, 2025 15:44
Copy link
Contributor Author

gavriliumircea commented Nov 6, 2025

Merge activity

  • Nov 6, 3:51 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Nov 6, 3:54 PM UTC: Graphite rebased this pull request as part of a merge.
  • Nov 6, 3:55 PM UTC: @gavriliumircea merged this pull request with Graphite.

@gavriliumircea gavriliumircea changed the base branch from feat/add-solana-chains to graphite-base/85 November 6, 2025 15:52
@gavriliumircea gavriliumircea changed the base branch from graphite-base/85 to main November 6, 2025 15:53
@gavriliumircea gavriliumircea force-pushed the feat/add-solana-contract-versions branch from d5dcae5 to b948242 Compare November 6, 2025 15:54
@gavriliumircea gavriliumircea merged commit 7c0afa0 into main Nov 6, 2025
2 of 3 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