Replies: 2 comments
-
|
Yes, feel free to create an issue about this. The main consideration I have is whether erroring would be a breaking change and if we have to wait until clap 4.0. |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
thanks! made the issue here: #3660 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I just filed a PR to starship to fix an issue where both
requiredandrequired_unless_presentwere used, as showed in the code below.Because
requiredis listed, therequired_unless_presentdoes not do anything, andnameis required no matter what. Am I right to assume that the combination of these two fields is always incorrect, and would it be possible to provide an error to the user if they do this?Beta Was this translation helpful? Give feedback.
All reactions