-
Notifications
You must be signed in to change notification settings - Fork 38.6k
Update RPC generate help for numblocks to include required #6631
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.