Remove DEPRECATED text, since it's just the default#9231
Conversation
ndeloof
left a comment
There was a problem hiding this comment.
👍
strictly speaking, we should handle --interactive=false and not just make this flag a No-Op, but I guess that's a corner case :D
|
Needs a rebase, @ulyssessouza |
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
18f9526 to
2902907
Compare
Yes, would be good to support that as well; also (from #9207 (comment)) probably produce an error when passing both |
|
Already added support for |
|
Yes, that one is more tricky; if the options do not "conflict", we could accept them, but if they "conflict", it's more tricky to define the behaviour (which one should take precedence??). In such cases, I usually make that an error-condition; the advantage with that is that you're sure nobody is able to use the combination (as it errors), which leaves the ability to "allow" the combination in future (if there's a decision on what the expected outcome is). |
|
indeed. I planned to rely on |
What I did
Remove DEPRECATED text, since it's just the default
Related issue
#9229 (comment)
(not mandatory) A picture of a cute animal, if possible in relation with what you did