Dustin Brickwood

Results 52 comments of Dustin Brickwood

Will wait until contracts base branch for v30 (`draft-v30`) is stable before completing. The branch used here initially is 2 months old.

@PatrickAlphaC Currently this is intentional as the installation uses the same base directory of `.foundryup`. If we want to avoid overwriting one another while also ensuring devs can execute the...

Related upstream PR that would close this: https://github.com/foundry-rs/foundry/pull/9505

Thanks for posting the issue. `forge coverage` is not fully supported but is being prioritized. I will keep this open in the meantime.

Thanks for posting @alphathis ! Apologies for the delayed response. We are currently in the process of reviewing how we are returning errors and I was able to reproduce this...

Thanks for posting @cytadela8 ! Apologies on the delay here. We currently do not have this planned and its something we currently do not support but perhaps we should considering...

This may be better suited for a Discussion post, but did not want it to live in an Notion doc so posted as an issue instead. CC: @popzxc @itegulov for...

Thanks for the feedback. Yes, much better approach to make use of a shell wrapper with macro usage. I started refactoring #578 to use a simplified version of Foundrys implementation....

Thanks for all the feedback team @sayon @itegulov @popzxc 💪 Definitely taking an iterative approach to the implementation, so I will open separate issues describing approaches for including `--gas-report`, `--vm-report`...

Related PRs: - https://github.com/matter-labs/anvil-zksync/pull/577 - https://github.com/matter-labs/anvil-zksync/pull/578