Stabilize DIA fork fixtures and RPC config#49
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
1307b9d to
308d04a
Compare
291ef84 to
9bd3682
Compare
Read Base RPC from BASE_RPC_URL in CI, repin fork goldens to FORK_BLOCK_BASE, drop debug console logging, and use LibDia.CHAIN_ID_BASE in tests. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Assert the exact AMZN price and timestamp at the fork-fixtures block. Co-authored-by: Cursor <cursoragent@cursor.com>
Treat empty CI secrets as missing and cover configured, local-default, and fail-closed URL selection paths. Co-authored-by: Cursor <cursoragent@cursor.com>
Exercise empty CI configuration through the pure resolver without mutating environment variables shared by other Forge suites. Co-authored-by: Cursor <cursoragent@cursor.com>
9bd3682 to
d1be19b
Compare
308d04a to
bcc10d5
Compare
2 participants
Summary
BASE_RPC_URLand fail closed in CI when unsetFORK_BLOCK_BASE(47365950) and sync test header docsconsole2fork logging and useLibDia.CHAIN_ID_BASEin testsscript/Build.solfor rainix copy-artifactsStack
Stacks on
audit/codegen-docs-imports. Formerly #37.Test plan
nix develop -c forge testCloses #12
Closes #27
Closes #30
Closes #31
Made with Cursor