cli icon indicating copy to clipboard operation
cli copied to clipboard

Published 20 hours ago • verified publisher icon urfave

Add App.InvalidFlagAccessHandler

Open icholy opened this issue 2 years ago • 4 comments

Follow up from #1434

icholy avatar Aug 02 '22 19:08 icholy

I couldn't test it myself, but I like this PR.

julian7 avatar Aug 08 '22 07:08 julian7

@julian7 I don't like the name. It sounds like it will get called when someone passes an unknown flag, ex mycli --unknown-flag

icholy avatar Aug 08 '22 13:08 icholy

@julian7 I don't like the name. It sounds like it will get called when someone passes an unknown flag, ex mycli --unknown-flag

Hmm, reasonable. What about UndefinedFlagHandler?

julian7 avatar Aug 08 '22 14:08 julian7

@julian7 I don't like the name. It sounds like it will get called when someone passes an unknown flag, ex mycli --unknown-flag

Hmm, reasonable. What about UndefinedFlagHandler?

That name has the same problem.

icholy avatar Aug 08 '22 14:08 icholy

@icholy lets leave it as it is. Thanks

dearchap avatar Aug 30 '22 22:08 dearchap