You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We should remove short options from args passed to the command.
Rationale
In some commands, we allow using unknown options. Unfortunately, after the recent migration to minimist, both the short option and its resolved equivalent are passed into the command. Historically, we've excluded only the resolved equivalents and not the short options from being added to the request. In the current state, the included shorts lead to request errors. Since we don't use them in our code, we should remove them from the args object passed into the command as well as the catch-all _ property.
The text was updated successfully, but these errors were encountered:
We should remove short options from args passed to the command.
Rationale
In some commands, we allow using unknown options. Unfortunately, after the recent migration to minimist, both the short option and its resolved equivalent are passed into the command. Historically, we've excluded only the resolved equivalents and not the short options from being added to the request. In the current state, the included shorts lead to request errors. Since we don't use them in our code, we should remove them from the args object passed into the command as well as the catch-all
_
property.The text was updated successfully, but these errors were encountered: