search

cgewecke / eth-gas-reporter

Gas usage per unit test. Average gas usage per method. A mocha reporter.
MIT License
603 stars 94 forks source link

Replace deprecated package request-promise-native #298

Closed ChristopherDedominici closed 1 year ago

ChristopherDedominici commented 1 year ago

The package request-promise-native is deprecated. I replaced it using the already present package sync-request.

cgewecke commented 1 year ago

@ChristopherDedominici Thanks!

After looking at this a bit I think I'm going to replace this with axios (or something) instead. I'd like to avoid locking the execution thread for these remote requests.

Will open a separate PR for that this afternoon and also resolve hardhat-gas-reporter 102