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.

Sign up for GitHub

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

Jump to bottom

EVM: Add uniswap test script #800

Merged
merged 9 commits into from
Sep 6, 2023
Merged

EVM: Add uniswap test script #800

merged 9 commits into from
Sep 6, 2023

Conversation

bkolad
Copy link
Member

@bkolad bkolad commented Sep 6, 2023 •
edited
Loading

Description

This PR introduces a hardhat script for testing uniswap-v2 functionality. The goal is to test this script with the EVM module.

Linked Issues

Testing

Manually tested with anvil.

Docs

Readme added.

@codecov
Copy link

codecov bot commented Sep 6, 2023 •
edited
Loading

Codecov Report

Merging #800 (13a0f97) into nightly (e4951ee) will increase coverage by 0.0%.
The diff coverage is n/a.

❗ Current head 13a0f97 differs from pull request most recent head d7497f4. Consider uploading reports for the commit d7497f4 to get more accurate results

see 2 files with indirect coverage changes

@bkolad bkolad marked this pull request as ready for review September 6, 2023 13:03
@bkolad bkolad requested a review from LukaszRozmej September 6, 2023 13:03
@bkolad bkolad changed the title WIP: uniswap script EVM: Add uniswap test script Sep 6, 2023
LukaszRozmej
LukaszRozmej approved these changes Sep 6, 2023
View reviewed changes
Copy link
Contributor

@LukaszRozmej LukaszRozmej left a comment

Choose a reason for hiding this comment

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

If it works then looks fine :)

@bkolad bkolad added this pull request to the merge queue Sep 6, 2023
Merged via the queue into nightly with commit 55d42b3 Sep 6, 2023
@bkolad bkolad deleted the blaze/uniswap_demo branch September 6, 2023 16:27
preston-evans98 pushed a commit that referenced this pull request Sep 14, 2023
@bkolad @preston-evans98
EVM: Add uniswap test script (#800)
ad81c7a 
* uniswap script

* cleanup

* npm fix

* update dependencies

* save contract addresses

* cleanup

* Update Readme.md

* Update Readme.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LukaszRozmej LukaszRozmej LukaszRozmej approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bkolad @LukaszRozmej