chore(deps): bump rain-lib-typecast 0.1.0 -> 0.1.4, rain-sol-codegen 0.1.0 -> 0.1.36 - #131
Conversation
…0.1.0 -> 0.1.36 rain-sol-codegen 0.1.36 spells forge-std as forge-std-1.16.2/ in vendored src this repo compiles (LibCodeGen, LibHexString), so forge-std moves 1.16.1 -> 1.16.2 in the same clean bump: foundry.toml, soldeer.lock and every versioned import rewritten; no alias remappings, no second copy. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 54 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (14)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe project updates dependency pins for ChangesDependency alignment
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This PR updates dependency versions and their import paths, including the required forge-std transitive update, without authored logic changes. The full test suite passes, and no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
The filter pinned the vendored dir by version; the forge-std bump moved it, so slither began analyzing forge-std's own Vm.sol (naming-convention hit on eth_getLogs). Same stale-pin prune as the rest of the bump. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
@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:
|
Clears the two stale deps in one PR:
rain-lib-typecast0.1.0 -> 0.1.4rain-sol-codegen0.1.0 -> 0.1.36 (the version raindex uses on main, so the API is proven in-org)Mechanics: versions bumped in
foundry.toml,soldeer.lockregenerated viaforge soldeer update, and every versioned import path rewritten (rain-lib-typecast-0.1.0/->-0.1.4/,rain-sol-codegen-0.1.0/->-0.1.36/).remappings.txtis gitignored and regenerated by soldeer in CI.Transitive forge-std bump (disclosed):
rain-sol-codegen0.1.36 hard-pinsforge-std-1.16.2/src/Vm.solin vendored src this repo compiles (LibCodeGen.sol,LibHexString.sol). Withrecursive_deps = falsethose imports resolve through this repo's remappings, so forge-std moves 1.16.1 -> 1.16.2 in the same clean bump:foundry.toml,soldeer.lock, and this repo's 12 forge-std imports rewritten. No alias remappings, no second forge-std copy. The only vendored files still spellingforge-std-1.16.1/are dep-internal test files this repo never compiles..gas-snapshotleft untouched: no CI job checks it, and the solmem bump merged yesterday (#130) set the same precedent for dep bumps.Verified locally in the repo's
#sol-shell:forge buildclean,forge test -vvvfull suite 118 passed / 0 failed / 0 skipped.QA
#sol-shell)foundry.toml/soldeer.lockand versioned import path rewrites only🤖 Generated with Claude Code
Summary by CodeRabbit
Chores
Tests