Skip to content
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

wip: Add Base Sepolia support #761

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

alex-stone
Copy link

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

feature

  • What is the current behavior? (You can also link to an open issue here)

Base Sepolia (84532) is an unsupported chain

  • What is the new behavior (if this is a feature change)?

Base Sepolia (84532) is a supported chain

  • Other information:

This depends on Uniswap/sdks#192 and will move out of Draft once that lands and a version is cut!

This follows up the work that @danielcrt did here, but on the new SDK monorepo:
Uniswap/sdk-core#110
#494

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.
This adds the Chain ID, Addresses, and WETH token configuration for
the Base Sepolia network (84532) to the smart order router.

Uniswap#494
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] Transitive: environment, eval, filesystem, network, shell, unsafe +538 360 MB

View full report↗︎

Copy link

Report is too large to display inline.
View full report↗︎

Next steps

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/[email protected] or ignore all packages with @SocketSecurity ignore-all

@@ -38,7 +38,7 @@
"@uniswap/default-token-list": "^11.13.0",
"@uniswap/permit2-sdk": "^1.3.0",
"@uniswap/router-sdk": "^1.14.0",
"@uniswap/sdk-core": "^5.9.0",
"@uniswap/sdk-core": "github:alex-stone/uniswap-sdks#alex-stone/add-base-sepolia",
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the submodule installation doesn't work nicely with the git / github syntax.

Will try to figure that out / hopefully land that PR!

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.

1 participant