chore: use dpp 4.0.0-RC2-SNAPSHOT for testnet and 2.0.6-SNAPSHOT for …#1499
Conversation
…feat/ctx-redeem-url-support
📝 WalkthroughWalkthroughThe Gradle script now maps Dash Platform SDK versions by product flavor, removes the defaultConfig ChangesFlavor-scoped Dash Platform versioning
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@wallet/build.gradle`:
- Around line 18-23: The dppVersions entries in the wallet build are pointing at
SNAPSHOT coordinates that the build cannot reliably resolve in clean CI. Update
the version selection used by the wallet build so `org.dashj.platform:dash-sdk`
points to released artifacts instead of the missing SNAPSHOTs, or ensure the
`dppVersions` values are backed by artifacts published to Sonatype or preloaded
into `mavenLocal()` before resolution. Locate the change in the `dppVersions`
map in the build script and adjust the version constants used for `prod`,
`_testNet3`, `staging`, and `devnet` accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
…mainnet
Issue being fixed or feature implemented
Related PR's and Dependencies
Screenshots / Videos
How Has This Been Tested?
Checklist:
Summary by CodeRabbit