build(sh): include BuildPointers.sol and forge fmt in build.sh#220
build(sh): include BuildPointers.sol and forge fmt in build.sh#220thedavidmeister wants to merge 1 commit into
Conversation
Currently build.sh only regenerates the meta CBOR artifact and stops. Regenerating src/generated/ERC4626Words.pointers.sol requires a separate manual step. Add a sol-shell block that runs BuildPointers.sol and forge fmt so a single ./script/build.sh regenerates all committed artifacts (meta/ + pointers + formatting) without extra manual steps. Closes #13 Co-Authored-By: Claude <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
Walkthrough
ChangesPointer regeneration in build.sh
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Summary
build.shonly regenerated the meta CBOR artifact;src/generated/ERC4626Words.pointers.solrequired a separate manual invocation offorge script script/BuildPointers.sol+forge fmt.build.shso a single./script/build.shregenerates all committed artifacts (meta CBOR + pointer constants + formatting) without extra manual steps.rain); the new pointer + fmt block runs in the sol-shell (needsforge).Closes #13
Co-Authored-By: Claude noreply@anthropic.com
Summary by CodeRabbit