+Ch0pin
+Ch0pin
> could you elaborate on why this is more desirable? Do you need it for publishing your crate? i think we just forgot to bump the ethers version in the...
Can you please give an example code which worked before / stopped working after?
+1 on not trying to redo all of Slither/SlithIR's work, but if we can get a reasonable set of features just by parsing with solang parser it'd be quite nice....
Per @mds1 suggestion let's go with the following 2 steps: 1 Add a broadcast overload that lets you specify the gas limit for the tx, `vm.broadcast(uint256 gasLimit)` and `vm.broadcast(address sender,...
> This one wasn't settled on, but I think we should change the broadcast overloads to specify margin on the gas estimate, instead of absolute limits. So if I'm estimating...
cc @onbjerg
@KittyFu307 `forge coverage --report lcov && genhtml lcov.info -o report --branch-coverage && open report/index.html` or just `forge coverage`!
> because the create2 address generation is in cli and well I cannot import it because of that. I'd refactor this to the `foundry-common` crate and import it from there!
That was the dapptools default I think. I'm open minded to either, but I personally like the optimizer for most use-cases that do not have to do with low level...
Can you share the script so we can reproduce this?