Cli: Possbile StringFlag Validation Bug

[IcE4k777]

Works as expected:
myApp subCommand --StringFlag myArgument

Works as expected, cli handles printing usage as no argument is given to --StringFlag:
myApp subCommand --StringFlag

Not working as expected:
myApp subCommand --StringFlag --myBoolFlag

It appears cli is just reading the next string after --StringFlag and considering that as the argument to --StringFlag, however, cli should print usage as no argument is really provided to --StringFlag. What do you guys think? Is this to much input validation for cli to deal with?

[jszwedko]

:+1: I could see this as a possible enhancement, but would require some preprocessing as the current behavior is due to how the flag standard library opts to handle it.

[stale[bot]]

This issue or PR has been automatically marked as stale because it has not had recent activity. Please add a comment bumping this if you're still interested in it's resolution! Thanks for your help, please let us know if you need anything else.

[stale[bot]]

This issue or PR has been bumped and is no longer marked as stale! Feel free to bump it again in the future, if it's still relevant.

[stale[bot]]

This issue or PR has been bumped and is no longer marked as stale! Feel free to bump it again in the future, if it's still relevant.

[stale[bot]]

This issue or PR has been bumped and is no longer marked as stale! Feel free to bump it again in the future, if it's still relevant.

[lynncyrin]

ok stale bot 😆

I marked this as help wanted 🙏

[stale[bot]]

This issue or PR has been automatically marked as stale because it has not had recent activity. Please add a comment bumping this if you're still interested in it's resolution! Thanks for your help, please let us know if you need anything else.

[xordspar0]

I'm not sure if my opinion counts for anything, but I prefer the current behavior. I wouldn't want cli to be too smart and try to figure out my intention. Maybe I really did want to pass --myBoolFlag as a string argument.

[stale[bot]]

This issue or PR has been bumped and is no longer marked as stale! Feel free to bump it again in the future, if it's still relevant.

[stale[bot]]

This issue or PR has been automatically marked as stale because it has not had recent activity. Please add a comment bumping this if you're still interested in it's resolution! Thanks for your help, please let us know if you need anything else.

[stale[bot]]

Closing this as it has become stale.