Skip to content

Fix search/replace typo in --no-TTY documentation#9329

Merged
ndeloof merged 1 commit intodocker:v2from
ericfreese:pseudo-notty-typo
Apr 4, 2022
Merged

Fix search/replace typo in --no-TTY documentation#9329
ndeloof merged 1 commit intodocker:v2from
ericfreese:pseudo-notty-typo

Conversation

@ericfreese
Copy link
Contributor

What I did

Commit abbba74 looks to have accidentally replaced pseudo-tty with pseudo-noTty in several places. In other places, it looks like it replaced pseudo-tty with pseudo-TTY, so I used the uppercased version in these places as well.

For example, running version 2.3.3, I get this output:

% docker-compose run --help

...

Options:
  ...
  -T, --no-TTY                Disable pseudo-noTty allocation. By default docker compose run allocates a TTY
  ...

I fixed what I believe were typos.

Related issue

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

@ndeloof
Copy link
Contributor

ndeloof commented Mar 31, 2022

Hi. For legal reason all commit need to be signed off.
Please amend your commit and force push your branch
git ci --amend --signoff && git push --force ...

Commit abbba74 looks to have
accidentally replaced `pseudo-tty` with `pseudo-noTty` in several
places. In other places, it looks like it replaced `pseudo-tty` with
`pseudo-TTY`, so I used the uppercased version in these places as well.

For example, running version 2.3.3, I get this output:

```
% docker-compose run --help

...

Options:
  ...
  -T, --no-TTY                Disable pseudo-noTty allocation. By default docker compose run allocates a TTY
  ...
```

Signed-off-by: Eric Freese <ericdfreese@gmail.com>
@ericfreese
Copy link
Contributor Author

Ok @ndeloof that should be done now.

@ndeloof ndeloof enabled auto-merge (rebase) April 4, 2022 17:02
@ndeloof ndeloof merged commit d871cb9 into docker:v2 Apr 4, 2022
@ericfreese ericfreese deleted the pseudo-notty-typo branch April 5, 2022 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants