-
Notifications
You must be signed in to change notification settings - Fork 3
trails rc-3 w/ multicall3
#87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Added new Trails contracts for rc-4 in README.md - Updated rc-4.yaml to include dependency on sequence_v3/rc_4 and modified creation code handling for TrailsRouter and TrailsRouterShim actions
…ce creation code handling
…4 contract addresses
rc-3 w/ multicall3 and rc-4 w/ sequence multicall3
rc-3 w/ multicall3 and rc-4 w/ sequence multicall3rc-3 w/ multicall3
…ails contracts for rc-3 deployment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds deployment configuration for trails release candidate 3 (rc-3) along with multicall3 dependency support. The rc-3 version of trails contracts now depends on and integrates with the multicall3 contract, which is being added as a new predeploy.
Key Changes
- Added multicall3 deployment configuration using ERC-2470 singleton factory
- Created rc-3 deployment configuration for trails contracts (TrailsRouter, TrailsRouterShim, TrailsIntentEntrypoint) with multicall3 integration
- Updated rc-1 and rc-2 configurations to declare multicall3 as a dependency
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| jobs/trails/rc-3.yaml | New deployment configuration for trails rc-3 that integrates multicall3 into TrailsRouter constructor |
| jobs/trails/rc-2.yaml | Added multicall3 dependency declaration |
| jobs/trails/rc-1.yaml | Added multicall3 dependency declaration |
| jobs/predeploy/multicall3.yaml | New deployment configuration for Multicall3 contract via ERC-2470 |
| README.md | Updated deployment addresses table with multicall3 and trails-rc-3 contract addresses |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
ScreamingHawk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
rc-3forf4c92b6a5ba5bb1d1c18e256a01a23ab1b5d238bat 0xsequence/trails-contracts@f4c92b6