Conversation
Pool manager contract ABI
Deleted due to naming fix
Deleted due to naming fix
Deleted due to naming fix
Deleted due to naming fix
Deleted due to naming fix
Naming fix
Liquidity pools list management class added, UniswapV4 improvements.
ErikBjare
left a comment
There was a problem hiding this comment.
Generally approved. Haven't checked why CI is failing yet.
Could use some auto-formatting by ruff.
|
Ah, I see CI is failing due to outdated actions. I think we can merge this and address the outdated CI and any CI issues related to this PR in follow-up. |
refactor
refactor
refactor
moved to util
moved to types
moved to constants
Structure refactoring; fixes; StateView contract support added
There was a problem hiding this comment.
Strange that this diff became so messy
ErikBjare
left a comment
There was a problem hiding this comment.
Nice, better structure!
I think it would be worthwhile to add a proper pre-commit/prek config with ruff and ruff-format though, would probably fix the weird diffs that suggest a lot more has changed than it has.
technical commit/fixes
pre-commit linter/formatter
done |
Basic functionality, like price fetching, quoting, and making simple swaps etc