Bump rain-deploy 0.1.7 -> 0.1.8 - #582
Conversation
`test/abstract/RainlangExpressionDeployerDeploymentTest.sol` imports `LibRainDeploy` under `rain-deploy-0.1.7/`. Consumers reach it through `OpTest` (raindex imports `rainlang-<v>/src/../test/abstract/OpTest.sol`), and raindex main pins `rain-deploy` 0.1.8; Soldeer installs one version per package name, so the published prefix has to be the one consumers pin. `forge soldeer update` regenerated `soldeer.lock` and `remappings.txt`; the superseded `rain-deploy-0.1.7/` line and dir are pruned. Every other pin already matches what raindex spells. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V8ViHcKLVk2YoS2joH4HdN
|
Local full suite on 6f86b1f through the repo flake, ARBITRUM_RPC_URL set for the fork tests: 225 suites, 1552 passed, 0 failed, 0 skipped. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe PR updates the Changesrain-deploy update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The dependency alignment is validated by the complete test suite, with no remaining merge-blocking risk identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
@coderabbitai assess this PR size classification for the totality of the PR with the following criterias and report it in your comment: S/M/L PR Classification Guidelines:This guide helps classify merged pull requests by effort and complexity rather than just line count. The goal is to assess the difficulty and scope of changes after they have been completed. Small (S)Characteristics:
Review Effort: Would have taken 5-10 minutes Examples:
Medium (M)Characteristics:
Review Effort: Would have taken 15-30 minutes Examples:
Large (L)Characteristics:
Review Effort: Would have taken 45+ minutes Examples:
Additional Factors to ConsiderWhen deciding between sizes, also consider:
Notes:
|
What
rain-deploy0.1.7 -> 0.1.8, re-spelled at the single import site (test/abstract/RainlangExpressionDeployerDeploymentTest.sol,LibRainDeploy). Lockfile and remappings regenerated byforge soldeer update; the supersededrain-deploy-0.1.7/line and dir pruned. No alias remapping.Why
raindex imports
rainlang-<v>/src/../test/abstract/OpTest.sol, which extendsRainlangExpressionDeployerDeploymentTest, which spellsrain-deploy-0.1.7/. raindex main pinsrain-deploy0.1.8 (#2865, Robinhood Chain + BNB Smart Chain) and Soldeer installs one version per package name, so with 0.2.4 installed raindex cannot resolve that import. The ruling on raindex is that mapping one version prefix onto another is not acceptable, so the fix is here, then a release; raindex #2863 pins the result.Every other pin here already matches what raindex spells (
forge-std1.16.2,@openzeppelin-contracts5.6.1,rain-solmem0.1.28,rain-sol-codegen0.1.36,rain-metadata0.1.7,rain-lib-typecast0.1.4,rain-math-float0.2.1,rain-math-float-deploy0.1.2,rainlang-interface0.2.8,rain-string0.2.0,rain-tofu-erc20-decimals0.1.1,rain-intorastring0.1.0,rain-lib-memkv0.1.0,rain-math-binary0.1.4,rain-extrospection0.1.14), so nothing else moves.rain-math-float-deploy0.1.2 still spellsrain-deploy-0.1.7/in itsRainDeploySuitesBase.sol; this repo only reaches itsLibDecimalFloatDeploy.sol, whose import closure (ErrDecimalFloat,LibLogTable) never touchesrain-deploy, so the bump compiles without arain-math-float-deployrelease.QA
forge buildexit 0 andforge fmt --checkclean through the repo flake;LibRainDeploy.sol0.1.7 -> 0.1.8 adds two chains and changes nothing the deployment test reads. Local full-suite result is in the first comment.🤖 Generated with Claude Code
https://claude.ai/code/session_01V8ViHcKLVk2YoS2joH4HdN
Summary by CodeRabbit