Added gasprice + gaslimit override params #117
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Added gasprice + gaslimit override params to the following commands:
Updated the
estimateGas
util with extra confirmations to safeguard against accidental overrides and wasting money on gas.Testing
Starting - Wallet Balance:
Normal call (no override gaspricegwei):
Command:
npx hardhat --network polygon deploy --name ERC721CM --symbol ERC721CMR2 --maxsupply 1000 --erc2198royaltyreceiver 0xa6da3705F53B516078AF138Bd99eF8708436981D --erc2198royaltyfeenumerator 500
Pre-run output:
Tx: 0x0649699cde118acd375d00317dd03a401044e92a75cbd02e5ae0afb146242773
Contract: 0x5f5B9c37ef920B349eCF100aBFf9AE3BEE4c6792
Actual output:
Updated call (using override gaspricegwei):
Command:
npx hardhat --network polygon deploy --name ERC721CM --symbol ERC721CMR2 --maxsupply 1000 --erc2198royaltyreceiver 0xa6da3705F53B516078AF138Bd99eF8708436981D --erc2198royaltyfeenumerator 500 --gaspricegwei 200 --gaslimit 5000000
Pre-run output:
Tx: 0x5dbb9f648cd3315f8effc40bfd8e8d3814bb3f632d91b5ae2af2c3a3db2ba093
Contract: 0xF038BBF62F05DB95e2f4a353B056Bc51F78B461f
Actual output: