Adds 'required' option to 'prompt_for_input' #237
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.
contribution under the same terms as this project's license, and
that I have the right to license my contribution under those terms.
cc: @mbland
This PR adds an extra argument to
@go.prompt_for_input
. If the function is called with this argument, eg@go.prompt_for_input 'variable' 'prompt' '' '' 'required'
and the user supplies no value, ie presses only enter, the prompt will reappear until the user actually supplies something. If instead ofrequired
another value is supplied, the behavior is the already existing one.The rational behind this PR is that as it stands now, using
@go.prompt_for_input
in a loop to force the user to actually submit sth, is exactly identical to usingread
directly. It saves nothing. With this PR, it only takes one line and you are sure that the user will provide some input. Moreover,@go.prompt_for_yes_or_no
already loops until the user supplies something. This PR makes the two functions more "feature compatible".