Skip to content
This repository has been archived by the owner on Mar 17, 2024. It is now read-only.

Bump ethers version #296

Merged

Conversation

ChristopherDedominici
Copy link
Contributor

Bump the ethers module version to remove warnings about deprecated packages during installation.

@ChristopherDedominici
Copy link
Contributor Author

ChristopherDedominici commented Sep 26, 2023

All the tests are passing locally except one. This test is also failing in the master branch so it's probably not related to my change. Maybe I have a wrong test configuration?
image

@cgewecke
Copy link
Owner

All the tests are passing locally except one. This test is also failing in the master branch so it's probably not related to my change. Maybe I have a wrong test configuration?

No, everything looks good.

Apologies, the "tests" here are really crazy - they're a fake mocha test suite which is meant to be visually inspected to make sure the output is formatted correctly.

In the fake suite there's a failing unit test which makes sure a table is printed even when hardhat unit tests fail.

@cgewecke cgewecke assigned cgewecke and unassigned cgewecke Sep 28, 2023
@cgewecke cgewecke self-requested a review September 28, 2023 18:46
Copy link
Owner

@cgewecke cgewecke left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@cgewecke cgewecke merged commit bf3d12c into cgewecke:master Sep 28, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants