-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Expose non-interactive option to CLI help #2404
Conversation
ae177a2
to
4e73766
Compare
Was actually under the impression it's |
Sorry. I double checked it a few minutes ago. Both of these don’t prevent prompts:
But this do:
Going to force push this change in a minute. |
Done. But I’m not sure if this feels right. Should it be:
instead of:
|
Is there anything I can do to help to get this merged? |
I was just hoping for some more opinions, as I understand your feeling that it doesn't seem entirely "right" using |
We could wait for a few days if there are other opinions especially about that comment above: #2404 (comment) |
72ee313
to
e0896d4
Compare
Any additional feedback? Will this get merged? Going to close this otherwise 😗 |
That’s it
Related to #1162
Having that option in the CLI help would have saved my a few minutes.