search

ApeWorX / ape-optimism

Optimism network provider plugin for the Ape Framework
https://www.apeworx.io/
Apache License 2.0
9 stars 9 forks source link

refactor: transaction decoding using the wrong field #11

Closed fubuloubu closed 1 year ago

fubuloubu commented 1 year ago

fixes: #7

fubuloubu commented 1 year ago

Need to test this with local network, I think there might be problems

antazoey commented 1 year ago

Need to test this with local network, I think there might be problems

I tested local networks on both main and this branch and I didn't detect any differences. However, reverts seem to fail right now with an integer overload error. I will create an issue for this.

I am also in the works of testing out the actual issue.

fubuloubu commented 1 year ago

Weird failure on previous CI pass

antazoey commented 1 year ago

Weird failure on previous CI pass

could it be the auto-cancel? If you push before the last one completes, it cancels the last ones now.