feat(cli scaffold): allow deprecated flags

[quentinalbertone]

Description

This Pr allows developers to deprecate flags in the cli

Checklist

(For exoscale contributors)

• Changelog updated (under Unreleased block)
• Testing

Testing

type instanceListCmd struct {
	exocmd.CliCommandSettings `cli-cmd:"-"`

	_ bool `cli-cmd:"list"`

	Zone v3.ZoneName `cli-short:"z" cli-usage:"zone to filter results to" cli-deprecated:"use another flag instead"`
}

$> go run github.com/exoscale/cli compute instance list -h
This command lists Compute instances.

Supported output template annotations: .ID, .Name, .Zone, .Type, .IPAddress, .IPv6Address, .State

Usage:
  exo compute instance list [flags]

Aliases:
  list, ls

Flags:
  -h, --help   help for list

Global Flags:
  -C, --config string            Specify an alternate config file [env EXOSCALE_CONFIG]
  -O, --output-format string     Output format (table|json|text), see "exo output --help" for more information
      --output-template string   Template to use if output format is "text"
  -Q, --quiet                    Quiet mode (disable non-essential command output)
  -A, --use-account string       Account to use in config file [env EXOSCALE_ACCOUNT]

$> go run github.com/exoscale/cli compute instance list --zone ch-gva-2
Flag --zone has been deprecated, use another flag instead
[ list of my instance here ]

$> go run github.com/exoscale/cli compute instance list -z ch-gva-2
Flag --zone has been deprecated, use another flag instead

[imiric]

Nice, thanks!

Could we also add a test for this in cmd_test.go?

[quentinalbertone]

Nice, thanks!

Could we also add a test for this in cmd_test.go?

That's a good point @imiric , I didn't check this file. I will try to add a unit test for this part of the code.

[kobajagi]

Looks good, thanks!

[pierre-emmanuelJ]

good with test, thanks