Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

ganache --miner.defaultTransactionGasLimit estimate fails with Error Cannot convert estimate to a BigInt #2398

Closed
davidmurdoch opened this issue Feb 17, 2022 · 3 comments · Fixed by #3233
Assignees

Comments

@davidmurdoch
Copy link
Member

defaultTransactionGasLimit cli coercion function needs to take the string "estimate" into account when attempting to parse the raw option value.

@MicaiahReid
Copy link
Contributor

@davidmurdoch, just to confirm, is this a typo on the title? ganache --miner.blockGasLimit should not accept "estimate" as an input value. Like your issue description says, we care about --miner.defaultTransactionGasLimit, right?

@davidmurdoch
Copy link
Member Author

Sounds like a typo. Maybe everything works fine?

@MicaiahReid
Copy link
Contributor

Nope, ganache --miner.defaultTransactionGasLimit estimate fails with the error specified above. (so does ganache --miner.blockGasLimit estimate, but I don't think we want that one to work)

@MicaiahReid MicaiahReid changed the title ganache --miner.blockGasLimit estimate fails with Error Cannot convert estimate to a BigInt ganache --miner.defaultTransactionGasLimit estimate fails with Error Cannot convert estimate to a BigInt Jun 14, 2022
@tenthirtyone tenthirtyone self-assigned this Jun 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants