-
Notifications
You must be signed in to change notification settings - Fork 5.4k
feat: Add support for Sepolia #6610
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
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Passing run #11581 ↗︎Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
|||||||||||||||
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #6610 +/- ##
==========================================
- Coverage 60.40% 59.48% -0.92%
==========================================
Files 732 732
Lines 21463 21473 +10
Branches 7091 7092 +1
==========================================
- Hits 12964 12773 -191
- Misses 8425 8624 +199
- Partials 74 76 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
|
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Pull request alert summary
📊 Modified Dependency Overview:
🚮 Removed packages: @uniswap/sdk-core@3.2.2 |
Sepolia is already added to |
@JFrankfurt Are you able to replicate? I can't seem to get the same behaviour for the same pool. |
cmcewen
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.
wooooot!
| const ARBITRUM_GOERLI_TICK_LENS_ADDRESSES = '0xb52429333da969a0C79a60930a4Bf0020E5D1DE8' | ||
|
|
||
| // sepolia v3 addresses | ||
| const SEPOLIA_V3_CORE_FACTORY_ADDRESSES = '0x0227628f3F023bb0B980b67D528571c95c6DaC1c' |
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.
not relevant to PR - but kinda surprised these don't live in the SDK
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.
This does feel like a process that could be part of the protocol deployment scripts to save a lot of manual entry.
| [SupportedChainId.GOERLI]: new Token(SupportedChainId.GOERLI, UNI_ADDRESS[5], 18, 'UNI', 'Uniswap'), | ||
| [SupportedChainId.MAINNET]: new Token( | ||
| SupportedChainId.MAINNET, | ||
| UNI_ADDRESS[SupportedChainId.MAINNET], |
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.
nice 👍


Description
11155111, important resources, and relevant contract addressesSlack thread:
Relevant docs:
Screen capture
Successful Pool

Successful swap

Test plan
QA (ie manual testing)