feat: demo script - Session 13#14
Merged
Merged
Conversation
…mo, reactive fixes
…x CI) The Sepolia fork suites bind to the live deployment and require a real RPC. CI has no .env, so setUp reverted with "vm.envString: environment variable SEPOLIA_RPC_URL not found", failing the Run tests step. Guard setUp to vm.skip(true) cleanly when the RPC is absent, so `forge test` passes in CI (278 pass, fork suites skip) while the fork tests still run locally where the RPC is configured. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ResetPoolTick.s.sol was added after the session's earlier per-file fmt pass, so its over-long PoolSwapTest.TestSettings line tripped CI's `forge fmt --check`. Run `forge fmt` makes the whole repo clean. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
`.githooks/pre-push` runs `forge fmt --check`, `forge build`, and `forge test` before every push, so the CI checks (fmt-check / build / test) can't fail on a push that passed locally. Enable per-clone with `make hooks` (sets core.hooksPath=.githooks); bypass a single push with `git push --no-verify`. The hook no-ops if forge isn't on PATH. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Session 13 complete. Sepolia fork integration tests, live end-to-end broadcast, RangeGuardDemo.s.sol, DemoLPRouter, and two Reactive Omni fork bug fixes.
What shipped
0x5eb9c8C0…Fee1(old0xC0e6…paused)0x62e2311b…462d88Docs