Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,8 @@ on:
env:
# The repo secret still points at a rate-limited Infura URL, so CI must use
# the public endpoint directly instead of preferring the secret on push runs.
PROVIDER_MAINNET: 'https://ethereum.publicnode.com'
#PROVIDER_MAINNET: 'https://ethereum.publicnode.com'
PROVIDER_MAINNET: 'https://eth.drpc.org'
PROVIDER_ARBITRUM: 'https://arb1.arbitrum.io/rpc'
PROVIDER_XDAI: 'https://rpc.ankr.com/gnosis'

Expand Down
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,12 @@ Contributors also earn this beautiful [GitPOAP](https://gitpoap.notion.site/What

## Changelog

_0.9.0_
* Added: Async versions of Uniswap V2-V4
* Added: V2 and V3 custom routes (untested)
* Added: Async pool cache service for Uniswap V4
* Removed: V1 support

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 V1 removal is misstated

The changelog says V1 support was removed, but Uniswap still defaults to version=1, accepts it, and loads V1 contracts. AsyncUniswap also accepts V1. This may lead readers to plan an unnecessary migration. If only V1 testing or maintenance ended, say that instead; otherwise remove this entry until the public clients match it.

Knowledge Base Used: Uniswap client SDK

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!


_0.8.0_

* Added: Uniswap V4 support
Expand Down
Loading