Support hardhat

[cgewecke]

A list of known issues:

• New provider design (see buidler 608)
• New artifact pipeline (see synthetix 739)

Update: a beta is available

npm install --save-dev hardhat-gas-reporter@beta

(It may be missing some data if your project contains shadowed contract names. See more below)

[alcuadrado]

Hey Chris!

Lot's of new things are coming out with hardhat. We'll have a migration guide for plugins in a few days.

We also took some time to adapt solidity-coverage. We'll send a PR soon. We are just finishing some details on how plugins are managed. Hardhat will be closer to the og version of Buidler, were plugins were just imported/required. This means that users won't need to deal with the type extensions manually.

[cgewecke]

@alcuadrado Ok fantastic, thanks so much. 🙂

[alcuadrado]

Hey @cgewecke, we have a migration guide for plugins now: https://github.com/nomiclabs/buidler-ts-plugin-boilerplate/blob/hardhat-migration/MIGRATION.md

There are @rc version of the hardhat plugins now. The main package is hardhat, and the plugins @nomiclabs/hardhat-*

[cgewecke]

@alcuadrado

Do you think it might make sense to make a new repo called hardhat-gas-reporter and deprecate this one?

[alcuadrado]

We are planning to rename the repos and leave a 1-line message explaining about the migration in the readme.

[mudgen]

@cgewecke @alcuadrado
How is adding support for hardhat coming along?

[cgewecke]

@mudgen Will look at it this weekend. Should be simple here...

[jmonteer]

hola!

any news on the support for hardhat?

thanks!

[cgewecke]

@jmonteer Doing it today!! Swear.

[mudgen]

@cgewecke You are the man!

[mudgen]

@cgewecke How is it going? I am curious. Is it easy or hard to port a buidler plugin to hardhat?

[cgewecke]

@mudgen

Really sorry - this is slightly trickier than I thought - I might have to modify the way eth-gas-reporter processes artifacts and publish a new version over there as well. Am working on this today and will get it out...

[alcuadrado]

Let me know if you need help with this, Chris!

[cgewecke]

@alcuadrado Thanks Patricio!

[cgewecke]

@mudgen There's a beta published now if you need this ASAP.

npm install --save-dev hardhat-gas-reporter@beta

If there are contracts that Hardhat needs a path-qualified name to load the artifact of, they'll be missing from the report.

This should only happen if the contract name is duplicated somewhere in the project (like in a node_modules dependency) or you're staging multiple versions of the same contracts in your project (like old/, new/)

[mudgen]

@cgewecke Great. Thank you.

[cgewecke]

This is out as hardhat-gas-reporter@1.0.0

Please just lmk if you run into problems with it. One thing that jumped out in e2e testing is that projects with qualified contract names have wide reports. (Perhaps over-wide, really)

Addressing that in a forth-coming patch at eth-gas-reporter and tracking the issue here in #40.