Matt Solomon
Matt Solomon
Any objections to closing this in favor of #3911 as the canonical issue to spec out some higher-level functionality over `forge script`? I think one issue to discuss such a...
Optimistically closing in favor of #3911 and https://github.com/foundry-rs/foundry/issues/4732
Everything here appears to be resolved now, so closing this
`polygon_mumbai` has been present for a long time, so this would be a breaking change that I'd like to avoid. Perhaps there's a way to cleanly support aliases though
Why do you need to delete the forks during execution of the forge script/test though? Is it a memory consumption issue?
In general the approach described here makes a lot of sense to me, definitely supportive. Regarding: > the number of samples and their randomness weight can be made configurable options...
I do agree with you that cast has a good amount of clutter in the commands and could benefit from a refactor. Before deprecating the mentioned commands, I think it'd...
Supportive of making this a meta issue, and agreed around being careful about breaking changes
Sharing a potential starting point for a refactor, largely courtesy of Claude. Very likely some commands are missing, and `cast convert` still needs fleshing out as mentioned above ``` cast...
Adding the Optimism codebase as another data point here, where it compiles with the default pipeline but not with the via-ir pipeline. Steps to reproduce: 1. Clone the repo: https://github.com/ethereum-optimism/optimism/...