"scw instance image update" should use a "positional argument" rather than an "image-id=" argument

[FredTreg]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Command attempted

scw instance image update 11111111-1111-1111-1111-111111111111 name=foo

Expected Behavior

I would expect this command to work without having to prefix the id with image-id=.
All other commands I could find (image get, server update, snapshot update) use an id argument without the something_id= prefix.

Actual Behavior

root ➜ .../coucou/infra/prod/app (main) $ scw instance image update 11111111-1111-1111-1111-111111111111 name=foo
Invalid argument '11111111-1111-1111-1111-111111111111': arg name must only contain lowercase letters, numbers or dashes

Hint:
Valid arguments are: image-id, name, arch, extra-volumes.{index}.id, from-server, public, tags.{index}, project, organization, zone

You could make this fix backward compatible by supporting both constructs.

More info

Version    2.16.1
BuildDate  2023-06-13T12:10:41Z
GoVersion  go1.20.4
GitBranch  HEAD
GitCommit  0bce8711
GoArch     amd64
GoOS       linux