Paul Berg

Results 723 comments of Paul Berg

> If your build takes a long time and you do multiple builds, you will see a lot of queues yeah I'm already starting to see the queues ..

Update: this seems to be about Solidity v0.8, not internal functions in libraries. I [just](https://github.com/paulrberg/solidity-template/commit/a2508538b2dfcf4106c7b47a7e7cbbd0f91589aa) added hardhat-gas-reporter to my solidity-template, and I am getting no gas report:

Re empty string in `excludeContracts` - oh, of course! Thanks for the heads-up. I removed the empty string and it works now. > The issue with libraries will require a...

> Feel free to edit this post. It looks like I don't have permission to edit it (perhaps because I'm not part of the `foundry-rs` org). I would suggest editing...

> Wen PRB Test powered by Forge Std? Well, I was hoping that Forge Std will be the one powered by PRBTest (as explained in https://github.com/foundry-rs/forge-std/issues/128). What I will do...

@gakonst none of the features and changes proposed for v0.3 imply a change of the pragma. I have opened a separate discussion to discuss that: https://github.com/foundry-rs/forge-std/issues/125.

> I don't think there is any compelling reason for us to bundle PRBTest in Forge Std instead of DSTest Building on top of the arguments I made in https://github.com/foundry-rs/forge-std/issues/128...

TLDR; I'm not adamant about switching from DSTest to PRBTest - in fact, I didn't even want to discuss this in the context of v0.3, it was @ZeroEkkusu who proposed...

> We should also plug the v0.3 branch into some of our own repos and make sure nothing (unexpected) breaks I've been using this branch for a few weeks now...

> potentially only supporting 0.8 for parseJson Sounds like something worth mentioning in https://github.com/foundry-rs/forge-std/issues/125!