Skip to content

Fix fish completion #8399

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

Closed
wants to merge 1 commit into from
Closed

Fix fish completion #8399

wants to merge 1 commit into from

Conversation

Chihsiao
Copy link

The fish completion shows subcommands improperly.

Before change

akira@home ~> docker-compose 
build                         (Build or rebuild services)  kill                           (Kill containers)  rm                  (Remove stopped containers)
config               (Validate and view the Compose file)  logs               (View output from containers)  run                     (Run a one-off command)
create                                  (Create services)  name       (specified in the client certificate)  scale  (Set number of containers for a service)
down                          (Stop and remove resources)  pause                           (Pause services)  start                          (Start services)
events         (Receive real time events from containers)  port  (Print the public port for a port binding)  stop                            (Stop services)
exec           (Execute a command in a running container)  ps                             (List containers)  top             (Display the running processes)
help                              (Get help on a command)  pull                       (Pull service images)  unpause                      (Unpause services)
images                                      (List images)  push                       (Push service images)  up                (Create and start containers)
in  (v3 files to their non-Swarm equivalent (DEPRECATED))  restart                       (Restart services)  version     (Show version information and quit)

in (v3 files to their non-Swarm equivalent (DEPRECATED)) at the last line is unexpected.

After change

akira@home ~> docker-compose 
build                  (Build or rebuild services)  logs               (View output from containers)  scale  (Set number of containers for a service)
config        (Validate and view the Compose file)  pause                           (Pause services)  start                          (Start services)
create                           (Create services)  port  (Print the public port for a port binding)  stop                            (Stop services)
down                   (Stop and remove resources)  ps                             (List containers)  top             (Display the running processes)
events  (Receive real time events from containers)  pull                       (Pull service images)  unpause                      (Unpause services)
exec    (Execute a command in a running container)  push                       (Push service images)  up                (Create and start containers)
help                       (Get help on a command)  restart                       (Restart services)  version     (Show version information and quit)
images                               (List images)  rm                   (Remove stopped containers)  
kill                             (Kill containers)  run                      (Run a one-off command)  

Signed-off-by: Akira Yamazaki <chihsiao@outlook.com>
@glours
Copy link
Contributor

glours commented Jul 27, 2022

Thanks for taking the time to create this issue/pull request!

Unfortunately, Docker Compose V1 has reached end-of-life and we are not accepting any more changes (except for security issues). Please try and reproduce your issue with Compose V2 or rewrite your pull request to be based on the v2 branch and create a new issue or PR with the relevant Compose V2 information.

@glours glours closed this Jul 27, 2022
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.

3 participants