Matt Solomon

Results 519 comments of Matt Solomon

That makes sense. Though I'm not sure I follow what you're doing, i.e. why do you have a prank before a broadcast? (I assumed you had the opposite, but yes...

Ah, I thought I'd seen unused variables in the past but perhaps [unused-state-variable](https://github.com/crytic/slither/wiki/Detector-Documentation#unused-state-variable) is what I was remembering. In which case, we can convert this into a feature request

Oh yes that would be very helpful! One of the issues with code scanning integration is that the local DB doesn't sync with what's been triaged in github, so running...

What if in the `[rpc_endpoints]` section of the config you try making the env var the full RPC URL instead of interpolating? Also, try `vm.rpcUrl("mainnet")` instead, which is what I...

You don't necessarily need to publish an NPM package for use with hardhat, e.g. forge-std is not published in npm but is still often used with hardhat. Adding a `package.json`...

References for invariant testing patterns, lmk if you have any questions when implementing: - https://github.com/foundry-rs/book/issues/497#issuecomment-1208473205 - https://github.com/mds1/solidity-sandbox/blob/main/test/4_InvariantNonceGoUp.t.sol

I agree that extending `foundry.toml` is the way to add config file support. Currently, doing so results in a forge warning, so I've created https://github.com/foundry-rs/foundry/issues/5866 to unblock config file support...

Closing as stale, we now have a spec to base this off of: https://specs.optimism.io/experimental/op-stack-manager.html

Hey @patitonar, thanks for the feedback! In the RFP "Total votable supply" is defined as "the total number of OP currently delegated to vote". The OP token self-delegates by default,...

@rappie What is your expected behavior here? My intuition is to agree with @ggrieco-tob that the corpus should still run with its original config. Are you suggesting to modify the...