build: raise language_version pragma to 0.23.0 - #598
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (48)
WalkthroughRaises ChangesCompact Toolchain Pragma Upgrade
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
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)
Comment |
Compiler 0.31.0 ships Compact language version 0.23.0. Raise the `pragma language_version` floor >= 0.21.0 -> >= 0.23.0 across all 47 .compact files so the declared minimum matches the toolchain. Depends on the toolchain upgrade in #597 (compiler 0.31.0). Verified: all contracts compile on 0.31.0 and the full vitest suite passes (1156 tests, 27 files).
b7a8fb3 to
1d5aa6b
Compare
Align ShieldedMultiSigV3 with the repo-wide pragma raised in OpenZeppelin#598 and with its own deps (Signer, Utils), which already declare >= 0.23.0. Refs: OpenZeppelin#604
Types of changes
No linked issue — follow-up to #597.
Raises the
pragma language_versionfloor>= 0.21.0→>= 0.23.0across all 47.compactfiles.0.23.0is the Compact language version that ships with compiler0.31.0(compactc-v0.31.0 release), so the declared minimum now matches the toolchain.Depends on #597
Stacked on the toolchain upgrade (#597), which bumps the compiler to
0.31.0. Until #597 merges, this PR's diff againstmainincludes the toolchain upgrade plus the pragma bump; once #597 lands, rebasing ontomaincollapses it to the pragma change only. Merge #597 first.Validation
0.31.0.PR Checklist
Further comments
Split out from #597 so the toolchain upgrade and the language-version floor land as separate, reviewable changes.
Summary by CodeRabbit