Matt Solomon

Results 519 comments of Matt Solomon

@0xPhaze @johnnyonline is this still an issue?

Just noting that one other part of compatibility would be to update [`StdChains`](https://github.com/foundry-rs/forge-std/blob/eb980e1d4f0e8173ec27da77297ae411840c8ccb/src/StdChains.sol#L152) and [`assumeNoPrecompiles`](https://github.com/foundry-rs/forge-std/blob/eb980e1d4f0e8173ec27da77297ae411840c8ccb/src/StdCheats.sol#L191-L221) in forge-std with the required Celo information

@Dendrimer I unassigned for you now, but if you do still plan to work on this let me know and I can reassign you

Yea the title is outdated, the current UX that was agreed on is a flag as described in this comment: https://github.com/foundry-rs/foundry/issues/2399#issuecomment-1212485018

Closing per the above comment, and this should be resolved now with recent parseJson updates/fixes

Closing as resolved per @DaniPopes' comment

There is now `envOr` to use a fallback value when an env var doesn't exist, so closing this

Agreed, using taplo to indent the keys would also help. I personally do that for all toml files and find it much more readable