Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fig up and attach should exit when processes exit #1

Closed
bfirsh opened this issue Jan 2, 2014 · 0 comments · Fixed by #165
Closed

fig up and attach should exit when processes exit #1

bfirsh opened this issue Jan 2, 2014 · 0 comments · Fixed by #165

Comments

@bfirsh
Copy link

bfirsh commented Jan 2, 2014

No description provided.

aanand added a commit that referenced this issue Mar 25, 2014
Closes #1 ヽ(*・ω・)ノ
dnephin referenced this issue in dnephin/compose Sep 23, 2014
oscarpicas referenced this issue in oscarpicas/fig Feb 4, 2015
chernjie referenced this issue in chernjie/compose Mar 23, 2015
Fix when pyyaml has interpreted line as a dictionary
yuval-k pushed a commit to yuval-k/compose that referenced this issue Apr 10, 2015
Closes docker#1 ヽ(*・ω・)ノ

Signed-off-by: Yuval Kohavi <yuval.kohavi@gmail.com>
shin- pushed a commit that referenced this issue Jan 18, 2018
Remove rebuild parameter ; set do_build instead

Signed-off-by: Ian Glen Neal <ian.gl.neal@gmail.com>
@rdissertori rdissertori mentioned this issue Jul 7, 2019
cstyles added a commit to cstyles/compose that referenced this issue May 1, 2020
The current method of generating completions for the fish shell relies
on running `docker-compose --help`. On my system (a 2018 MacBook Pro)
this routinely takes several hundred milliseconds:

```
$ hyperfine --warmup 3 'docker-compose --help'
Benchmark docker#1: docker-compose --help
  Time (mean ± σ):     351.6 ms ±  16.0 ms    [User: 295.5 ms, System: 44.3 ms]
  Range (min … max):   332.8 ms … 384.0 ms    10 runs
```

If I configure fish to automatically load these completions, it makes
launching a new fish process very sluggish.

In addition, the regular expression used to parse the output of
`docker-compose --help` incorrectly generates completions for `name` and
`in` due to the following lines:

https://github.com/docker/compose/blob/9c5351cf27d10a7b8078496fb21a37929b6687cb/compose/cli/main.py#L208
https://github.com/docker/compose/blob/9c5351cf27d10a7b8078496fb21a37929b6687cb/compose/cli/main.py#L212

This commit fixes these issues by explicitly listing subcommands instead
of generating them dynamically. This means that adding/removing
subcommands won't automatically update the completions but according to
the git blame for `compose/cli/main.py`, no one has touched the list of
subcommands in the help message in 3+ years so I hope that won't be an
issue.

Signed-off-by: Collin Styles <collingstyles@gmail.com>
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 a pull request may close this issue.

1 participant