fix(hammers/jackhammer): docker no-TTY option capitalization

[tristpinsm]

docker cares about case

[BrianJKoopman]

Hmm, strange, the -h help shows it lowercase:

$ docker compose run -h
Flag shorthand -h has been deprecated, use --help
Usage:  docker compose run [OPTIONS] SERVICE [COMMAND] [ARGS...]

Run a one-off command on a service

Options:
      --build                       Build image before starting container
      --cap-add list                Add Linux capabilities
      --cap-drop list               Drop Linux capabilities
  -d, --detach                      Run container in background and print container ID
      --dry-run                     Execute command in dry run mode
      --entrypoint string           Override the entrypoint of the image
  -e, --env stringArray             Set environment variables
      --env-from-file stringArray   Set environment variables from file
  -i, --interactive                 Keep STDIN open even if not attached (default true)
  -l, --label stringArray           Add or override a label
      --name string                 Assign a name to the container
      --no-deps                     Don't start linked services
  -T, --no-tty                      Disable pseudo-TTY allocation (default: auto-detected)

What version of compose are you using?

[tristpinsm]

this smurf server is using

cryo@smurf-so8-lat:~$ docker compose version
Docker Compose version v2.18.1

and expects a capitalized flag (fails with the lower-case version)

cryo@smurf-so8-lat:~$ docker compose run -h
Flag shorthand -h has been deprecated, please use --help

Usage:  docker compose run [OPTIONS] SERVICE [COMMAND] [ARGS...]

Run a one-off command on a service.

Options:
      --build                 Build image before starting container.
  -d, --detach                Run container in background and print container ID
      --dry-run               Execute command in dry run mode
      --entrypoint string     Override the entrypoint of the image
  -e, --env stringArray       Set environment variables
  -i, --interactive           Keep STDIN open even if not attached. (default true)
  -l, --label stringArray     Add or override a label
      --name string           Assign a name to the container
  -T, --no-TTY                Disable pseudo-TTY allocation (default: auto-detected).

[tristpinsm]

apparently this was resolved in v5.3.0

[BrianJKoopman]

Yeah, I'm on 5.4.0. What a silly thing. Sorry to do this, but let's just go back to -T where it's invoked. I'll make an inline suggestion for the docstring.

[tristpinsm]

agreed, that makes the most sense (and this is a very silly thing)

[BrianJKoopman]

Looks good, thanks!