feat: add Args.option() for typed arg options#1568
Conversation
|
Updated to include stricter type parsing: I feel it is easy to assume that options like I think theoretically we could make |
|
@jfeingold35 Any thoughts on this? |
|
@rexxars , sorry for the delay! I'll get this reviewed shortly and talk it over with the team. |
|
@rexxars , the code looks clean to me. We'll do some QAing and then we can get this approved and merged. However, it would be ideal if you could have a few runtime tests, e.g.,
|
|
Will have a look. I couldn't find any documentation on |
Adds a new
Args.option()method mirroringFlags.option(), which provides type-safe option usage:Setting it as
requiredremoves theundefinedfrom the union.