Bump rain-sol-codegen 0.1.37, rainlang-interface 0.2.9, rain-deploy 0.1.10 - #583
Conversation
….1.10 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V8ViHcKLVk2YoS2joH4HdN
|
Important Review skippedToo many files! This PR contains 256 files, which is 156 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (256)
You can disable this status message by setting the 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
Bumps three soldeer pins together in
foundry.toml:rain-sol-codegen0.1.36 -> 0.1.37rainlang-interface0.2.8 -> 0.2.9rain-deploy0.1.8 -> 0.1.10 (0.1.9 was never published)soldeer.lockandremappings.txtfollow, and every versioned import path undersrc/,script/andtest/is rewritten to the new package directories. Exactly one copy of each package remains independencies/andremappings.txt; nothing is aliased.script/Build.solwas re-run andforge fmtapplied;src/generated/RainlangReferenceExternPointers.solis unchanged by the bump.One incidental fix so
forge build --deny warningsclears the compiler stage: the twoLibInterpreterStateDataContract.t.solfuzz tests that only callvm.assume/boundare nowpure, which solc 0.8.25 already warned about on main. The 21 pre-existingforge lintboolean-cstfindings are untouched; none is on a line this bump changes.Why
The org health dashboard flags the pins as stale. The three move together because
rainlang-interface0.2.9 exists only to re-pin its ownrain-sol-codegenimport to 0.1.37, so bumping either alone would leave two codegen versions in the tree.No API adaptation was needed:
rain-sol-codegen0.1.37 differs from 0.1.36 only in its README.rainlang-interface0.2.9 differs from 0.2.8 only in theLibGenParseMetaimport ofrain-sol-codegen.rain-deploy0.1.10 changesAddressRegistryroot,MigrationRegistry/IMigrationRegistryV2and addssrc/generated/0_1_10;LibRainDeploy, the only thing this repo imports from it, is byte-identical.QA
🤖 Generated with Claude Code
https://claude.ai/code/session_01V8ViHcKLVk2YoS2joH4HdN