zerosnacks comments

Results 376 comments of


                                            zerosnacks

test `test_json_roundtrip_guessed` failed

@DaniPopes do you have an approach / fix in mind?

Tracking: Use Solar for Solidity AST

Hi @MathisGD, it is currently part of our 1.1 milestone and in focus - `fmt` is quite large / complex unfortunately, I wouldn't consider it a good first issue. In...

feat: add support to utility feature like npkill to remove `/libs` dependencies in a non-destructive way

Hi @StErMi thanks for your suggestion In my opinion this is out of scope for Foundry and I'm hesitant to implement a tool that has the ability to delete directories....

feat: add support to utility feature like npkill to remove `/libs` dependencies in a non-destructive way

Closing as `not planned` per discussion above - thanks for your suggestion!

feat(config): enable dynamic test linking by default

I got feedback from @emo-eth at Frontiers that dynamic test linking does not always work yet and if he is able to get a reproducible he will open a ticket...

fix(forge config): set default evm version

Related: https://github.com/foundry-rs/foundry/issues/8614

fix(forge config): set default evm version

pending review cc @yash-atreya / @klkvr

Support cast run quorum private txn

Personally not a fan as it is highly vendor specific and therefore I would be hesitant to add this; not because of the proposed functionality of implementation but whenever we...

feat(cheatcodes): extend usable range in `assertApproxEqRel`

cc @klkvr would you mind double checking this proposed change

bug(`cheatcodes`): expectEmit behaves weirdly

Hey @MathisGD, thanks for the report - I agree, it would be expected that this reverts with `EvmError: Revert` Related section: https://github.com/foundry-rs/foundry/blob/292d248db6f7a1fd0279e9892e566ef2674332ae/crates/cheatcodes/src/inspector.rs#L1645-L1665