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
Enforce that options (.e.g. --watch) must come last in our CLI.
Currently you can do stuff like hsm --watch push. This is ambiguous because it makes it look like you're trying to do hsm --watch=push. Although we don't accept option values not joined to the option name via =, other command applications do. Our users may mistake hsm for a command that does. We would then misinterpret that argument that was intended for an option as either a command or a command argument which could cause unexpected behaviour.
Deprecate it starting 0.22
Enforce it starting 0.23
The text was updated successfully, but these errors were encountered:
SarahIsWeird
added a commit
to SarahIsWeird/hackmud-script-manager
that referenced
this issue
Feb 3, 2025
Enforce that options (.e.g.
--watch
) must come last in our CLI.Currently you can do stuff like
hsm --watch push
. This is ambiguous because it makes it look like you're trying to dohsm --watch=push
. Although we don't accept option values not joined to the option name via=
, other command applications do. Our users may mistakehsm
for a command that does. We would then misinterpret that argument that was intended for an option as either a command or a command argument which could cause unexpected behaviour.The text was updated successfully, but these errors were encountered: