Matt Solomon
Matt Solomon
Do you know if this was always the case, or was there a regression in foundry or forge-std? The reverting values are actually 20 bytes long (40 hex chars), so...
Probably because the NFT is a proxy contract, IIRC StdStorage does not support that currently and requires the slot to be in the `_target` address
Ideally we could add proxy support to StdStorage, though I'm not sure offhand how much work that would be. Might not be too bad if we only support standard proxy...
Mainnet USDC recently ugpraded and now stored balance in a packed slot, so `deal` for it will be fixed once https://github.com/foundry-rs/forge-std/pull/505 is merged
Correct, you will need to upgrade your forge-std dependency to the latest release, not just your foundry version
Tracking in https://github.com/foundry-rs/foundry/issues/3801, cc @odyslam
Once this is merged, we will open a new PR that causes kontrol to fail, and manually dispatch the kontrol proofs. This should trigger an alert to slack if everything...
Thanks for the details! For chains not present in the default list, you should be able to use the `setChain` method here to define new chains. Adding this makes both...
Notes: - Gas scheduling is complex and it will be hard to cover everything perfectly. - #67 may include opcode level gas tests as well. - Minimal useful implementation of...
That's true. I plan to refactor things a bit so each chain has an input file, and 1559 parameters could be an input. But then the question becomes: What to...