Uniswap
chore: bump sor to 4.0.0 - feat(breaking): top-level alpha router native currencyIn, currencyIn, quoteCurrency support
Release https://github.com/Uniswap/smart-order-router/pull/721.
I ran all the e2e tests against local routing-api instance, and saw that the cached routes hit rate is still significantly higher than miss rate, which gives me the confidence the change is working as expected:
-
#844
👈 -
main
This stack of pull requests is managed by Graphite. Learn more about stacking.
Join @jsy1218 and the rest of your teammates on Graphite
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
| Package | New capabilities | Transitives | Size | Publisher |
|---|---|---|---|---|
| npm/@uniswap/[email protected] | Transitive: environment, eval, filesystem, network, shell | +109 |
119 MB |
🚮 Removed packages: npm/@uniswap/[email protected]
![socket-security[bot] avatar](https://avatars.githubusercontent.com/in/156372?v=4 "Published by a pub.dev verified publisher"){kind=small}
Graphite Automations
"Request reviewers once CI passes on routing-api repo" took an action on this PR • (09/24/24)
4 reviewers were added and 1 assignee was added to this PR based on 's automation.
![graphite-app[bot] avatar](https://avatars.githubusercontent.com/in/158384?v=4 "Published by a pub.dev verified publisher"){kind=small}
Shall we add some tests for the v4 case?
potentially unit test. We aren't able to add native currency routing e2e test, because we don't have a native currency pool in sepolia pool manager.
Update - just added native currency routing cached routes unit test @cgkol