Skip to content

Commit

Permalink
add some logging
Browse files Browse the repository at this point in the history
  • Loading branch information
amiecorso committed Oct 4, 2023
1 parent 1afa195 commit 965c202
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions plugins/fireblocks/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ extendConfig(
(config: HardhatConfig, userConfig: Readonly<HardhatUserConfig>) => {
const userNetworks = userConfig.networks;
if (userNetworks === undefined) {
return;
throw new Error('No networks defined in hardhat config.');
}
for (const networkName in userNetworks) {

Check warning on line 29 in plugins/fireblocks/index.ts

View workflow job for this annotation

GitHub Actions / lint

Using 'ForInStatement' is not allowed

Check warning on line 29 in plugins/fireblocks/index.ts

View workflow job for this annotation

GitHub Actions / lint

The body of a for-in should be wrapped in an if statement to filter unwanted properties from the prototype
const network = userNetworks[networkName]! as HttpNetworkUserConfig;

Check warning on line 30 in plugins/fireblocks/index.ts

View workflow job for this annotation

GitHub Actions / lint

Forbidden non-null assertion
Expand All @@ -49,7 +49,8 @@ extendConfig(
);

extendEnvironment((hre) => {
if ((hre.network.config as HttpNetworkUserConfig).fireblocks != undefined) {
if ((hre.network.config as HttpNetworkUserConfig).fireblocks !== undefined) {
hre.log(`Using Fireblocks signer for network ${hre.network.name}...`);
const httpNetConfig = hre.network.config as HttpNetworkUserConfig;
const eip1193Provider = new HttpProvider(
httpNetConfig.url!,
Expand Down

0 comments on commit 965c202

Please sign in to comment.