Matt Solomon

Results 519 comments of Matt Solomon

Yes, thank you! Just assigned you :)

This was completed by https://github.com/ethereum-optimism/superchain-ops/pull/266, thank you @ShubhSensei!

Can you check what forge version they are on? My guess is they're on an old foundry version that has a bug in the cheat where an empty string is...

I believe this was due to the foundry version, so closing this issue, and we can investigate if we come across this again

Being able to sign from the UI would be a nice way to easily support a lot of different wallets, so I'd like to keep this open and think about...

`forge inspect abi` will give you all this information, we just need to add support to convert it into a table format by appending the `--pretty` flag. Similar to `forge...

This would be a pretty sweet feature. Huge bonus points if you can evaluate state variables by name from the source code too, like you can with tenderly debugger (both...

Agree with everything here. Only comment is on @slgraham third item: Perhaps instead of `extra-watch-dirs`, we just put those additional docs in the same `docs` folder that gets created by...

Solidity's output does not return the max length of the enum, so the fuzzer can't know the max value to use. As a workaround for now you should pass a...

This is because of recent `expectRevert` changes—you must refactor to only use `expectRevert` on calls. When you do `uint256 price = LibChainlink.price(address(0), type(uint256).max, 0);`, since `price` is an internal method,...