soldeer migration#18
Conversation
Mirrors rain.solmem and the level-1 leaves. Closes (the soldeer issue on rain.datacontract). Dependencies migrated: - forge-std → forge-std-1.16.1 (soldeer) - rain.solmem → rain-solmem-0.1.3 (soldeer) Source-side: rewrote 4 imports across LibDataContract.sol and the test file from submodule paths to soldeer paths. The `..//../lib/...` relative path that sneaked in for LibPointer is gone — soldeer remappings make the path explicit. CI: drop the bespoke matrix workflow; use rainix's static, legal, and test reusables (and the publish wrapper). soldeer.lock pinned. Verified: forge test passes (6/6). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (17)
WalkthroughThis PR migrates the repository from Git submodule-managed dependencies to Soldeer package management. It removes submodule declarations, establishes foundry.toml dependency configuration, updates all imports to versioned paths, refactors GitHub Actions workflows into modular reusable tasks, simplifies flake.nix outputs, and restructures documentation and project configuration files accordingly. ChangesSoldeer Package Management Migration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related issues
✨ 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 |
Migrate from git submodules to soldeer.
Dependencies:
CI: drop the bespoke matrix workflow; use rainix's static, legal, and test reusables (plus the publish wrapper). soldeer.lock pinned.
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation
Chores