[ ] π Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] π Bug fix (a non-breaking change that fixes an issue)
[ ] π Enhancement (improving an existing functionality like performance)
[ ] β¨ New feature (a non-breaking change that adds functionality)
[ ] π§Ή Chore (updates to the build process or auxiliary tools and libraries)
[ ] β οΈ Breaking change (fix or feature that would cause existing functionality to change)
π Description
Resolves #148
I noticed that citty didn't provide null-safety on context.args even strict is true, this will break some type checks. In this pr I make all args be nullable first, then make required and items with default values ββnon-nullable.
And ParsedArgs type has some errors when parsing enum, it just return the options array itself but the correct should be the type of items in the array. In this pr I just simplly fixed it, but the best fix should make the type more precise.
π Checklist
[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly. (N/A)
π Linked issue
148
β Type of change
π Description
Resolves #148
I noticed that citty didn't provide null-safety on
context.args
evenstrict
istrue
, this will break some type checks. In this pr I make all args be nullable first, then make required and items with default values ββnon-nullable.And
ParsedArgs
type has some errors when parsing enum, it just return theoptions
array itself but the correct should be the type of items in the array. In this pr I just simplly fixed it, but the best fix should make the type more precise.π Checklist