ApeWorX / ape-foundry

Foundry network provider for the Ape Framework
https://www.apeworx.io/
Apache License 2.0
33 stars 13 forks source link

fix: update web3 isConnected() #19

Closed ghost closed 1 year ago

ghost commented 1 year ago

This is needed as a companion to https://github.com/ApeWorX/ape/pull/1082 since the plugin uses ape's web3 dependency

antazoey commented 1 year ago

Can we fix the mypy issue?

antazoey commented 1 year ago

Oh I see, it require ape release. Perhaps let's keep these as drafts until Ape releases

ghost commented 1 year ago

Oh I see, it require ape release. Perhaps let's keep these as drafts until Ape releases

Okay, I was about to ask since the test failures are from the same issue

ghost commented 1 year ago

Merging this to an upstream branch so that tests and secrets can work 👀