Expand cli documentation relating to the no-input flag (#1543)#1587
Expand cli documentation relating to the no-input flag (#1543)#1587insspb merged 3 commits intocookiecutter:masterfrom
Conversation
|
Hey @jeremyswerdlow thanks for taking time for this, and sorry for the delay on the review! While we're at it, I'd like to be able to fix other problems with flag's documentation within the codebase. I tracked down all the places where
In all those places the docstring just says "Prompt the user at command line for
"Suppress any prompts" doesn't say anything about how the function
Same as above, how does the function behave if the flag is
We should document also here the fact that Would you mind taking care of them? |
|
Hey @jeremyswerdlow are you still interested on working on this? I don't want to put pressure, it's just to know if I should wait or add those changes by myself and merge the pr. Update (2021-10-26) |
|
@jeremyswerdlow @glumia Thank you for contribution, will be in next release. |
Addresses the concerns in #1543 that not all side effects are clearly documented