Update RPC generate help for numblocks to include required#6631
Update RPC generate help for numblocks to include required#6631laanwj merged 1 commit intobitcoin:masterfrom ChainQuery:master
Conversation
The `generate` RPC has no default `numblocks` and a numeric value is required.
|
Re-submission of #6609 - sorry for the double request. |
|
@laanwj Continuing from #6609. Being explicit is nice goal. But in this case, setting default to 1 is much better because regtest/testnet is being used by people who should know what they are doing. When calling Setting the default to one can speed their work (or at least mine), as usually one block is enough. But if you really NACK setting the default to 1, my ACK here, of course. No problem with such decision. |
|
@paveljanik I agree that when calling I like the idea of being explicit in this case, just my 0.02 bits... |
|
@ChainQuery after forgetting that miner won't probably mine his blocks using But as I said above, I can live with being explicit. But the UX lecture told me to do something when you can expect something to happen and you can't broke anything with it. |
|
@paveljanik For a user-facing UI I'd probably have agreed. But this is an API. Let's not have a bikeshedding discussion about a call that's only used for testing. It's not like adding '1' in a test script is a big drain on anyone's time. |
e83df07 Update RPC generate help for numblocks to include required (Ian T)
The `generate` RPC has no default `numblocks` and a numeric value is required. Github-Pull: bitcoin#6631 Rebased-From: e83df07
Bitcoin 0.12 cleanup PRs 2 Cherry-picked from the following upstream PRs: - bitcoin/bitcoin#6631 - bitcoin/bitcoin#6664 - Only the first commit (we already had the second through bitcoin/bitcoin#6825). - bitcoin/bitcoin#6669 - bitcoin/bitcoin#6887 - Only the non-QT parts. - bitcoin/bitcoin#6962 - bitcoin/bitcoin#6822 - Only first and third commits (we already had the second through an earlier PR). - bitcoin/bitcoin#7136 - Excludes Travis CI changes, and fixes to documents we don't have anymore. - bitcoin/bitcoin#7084 - bitcoin/bitcoin#7509 - bitcoin/bitcoin#7617 - bitcoin/bitcoin#7726 Part of #2074.
The
generateRPC has no defaultnumblocksand a numeric value is required.