Skip to content

Conversation

@alex-stone
Copy link

@alex-stone alex-stone commented Nov 5, 2024

Description

This adds the Chain ID, Addresses, and WETH token configuration for the Base Sepolia network (84532).

This uses the addresses referenced in the Base Deployments doc: https://docs.uniswap.org/contracts/v3/reference/deployments/base-deployments

This also uses this as the source for the WETH token on base sepolia: https://sepolia.basescan.org/token/0x4200000000000000000000000000000000000006

This is following up on the changes that were being made on the old SDK repos:
Uniswap/sdk-core#110
Uniswap/smart-order-router#494

How Has This Been Tested?

This has not been explicitly tested yet (outside of CI), will be utilizing this in the context of Uniswap/smart-order-router.

Are there any breaking changes?

This should be purely additive constant configurations.

(Optional) Follow Ups

[Things that weren't addressed in this PR, ways you plan to build on this work, or other ways this work could be extended]
Plan to extend this to smart-order-router similar to: Uniswap/smart-order-router#494

This adds the Chain ID, Addresses, and WETH token configuration for
the Base Sepolia network (84532).

This uses the addresses referenced in the Base Deployments doc:
https://docs.uniswap.org/contracts/v3/reference/deployments/base-deployments

This also uses this as the source for the WETH token on base sepolia:
https://sepolia.basescan.org/token/0x4200000000000000000000000000000000000006

This is following up on the changes that were being made on the old
SDK repos:
Uniswap/sdk-core#110
Uniswap/smart-order-router#494
@alex-stone alex-stone requested review from a team and jsy1218 as code owners November 5, 2024 22:37
alex-stone added a commit to alex-stone/smart-order-router that referenced this pull request Nov 5, 2024
This bumps the `sdk-core` to pull in configuration changes made in:
Uniswap/sdks#192

This will be replaced by a standard version bump once that PR lands
properly.
@jsy1218
Copy link
Member

jsy1218 commented Jan 6, 2025

looks like github CI has to run from the Uniswap/sdks repo, not a forked repo. I created a PR to expedite supporting Base Sepolia #228

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants