Fix help output for "exec --no-tty" option

[tonyo]

What I did

Replaced backticks with single quotes in the help output of docker compose exec for the --no-TTY option.

Currently the output of docker compose exec --help looks like this:

✘ $ docker compose exec --help
Usage:  docker compose exec [OPTIONS] SERVICE COMMAND [ARGS...]

Execute a command in a running container

Options:
  -d, --detach                       Detached mode: Run command in the background
      --dry-run                      Execute command in dry run mode
  -e, --env stringArray              Set environment variables
      --index int                    Index of the container if service has multiple replicas
  -T, --no-TTY docker compose exec   Disable pseudo-TTY allocation. By default docker compose exec allocates a TTY.
      --privileged                   Give extended privileges to the process
  -u, --user string                  Run the command as this user
  -w, --workdir string               Path to workdir directory for this command

Note the -T line:

  -T, --no-TTY docker compose exec   Disable pseudo-TTY allocation. By default docker compose exec allocates a TTY.

That "docker compose exec" part after --no-TTY is most likely unintentional; it's there because Cobra (or more specifically, pflag) found the backticks and extracted the wrapped value via UnquoteUsage function.
This is a not-so-well documented behavior, see e.g. this and this.

After the change the output looks as expected:

✓ $ ./bin/build/docker-compose exec --help
Usage:  docker compose exec [OPTIONS] SERVICE COMMAND [ARGS...]

Execute a command in a running container

Options:
  -d, --detach            Detached mode: Run command in the background
      --dry-run           Execute command in dry run mode
  -e, --env stringArray   Set environment variables
      --index int         Index of the container if service has multiple replicas
  -T, --no-tty            Disable pseudo-TTY allocation. By default 'docker compose exec' allocates a TTY.
      --privileged        Give extended privileges to the process
  -u, --user string       Run the command as this user
  -w, --workdir string    Path to workdir directory for this command

Note

The By default 'docker compose exec' allocates a TTY. part of the output is not entirely correct, given that the default value of the flag is !dockerCli.Out().IsTerminal() (source). It'd be more like, "By default 'docker compose exec' allocates a TTY when the output is a terminal."
Let me know if that sounds better and I'll update it in a separate PR.

(not mandatory) A picture of a cute animal, if possible in relation to what you did

[ndeloof]

For legal reasons, contributor MUST sign-off commits before we can merge. Please amend your commit with git commit --signoff --amend and force push

[ndeloof]

Nice catch by the way, this surprising feature in pflags we probably could use to improve help messages in a few places

[glours]

LGTM

[glours]

@tonyo You need to run make docs to update the documentation files

[tonyo]

You need to run make docs to update the documentation files

Done 👍

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!