Skip to content

Add zsh completion for 'service {create,update} --group-{add,rm}'#26478

Merged
vdemeester merged 1 commit intomoby:masterfrom
sdurrheimer:zsh-completion-service-group-add-rm
Sep 13, 2016
Merged

Add zsh completion for 'service {create,update} --group-{add,rm}'#26478
vdemeester merged 1 commit intomoby:masterfrom
sdurrheimer:zsh-completion-service-group-add-rm

Conversation

@sdurrheimer
Copy link
Contributor

ref. #25317

@albers Thx for the ping

Signed-off-by: Steve Durrheimer [email protected]

@cpuguy83
Copy link
Member

LGTM

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this correct? --group-add is available for both docker service update and docker service create but --group-rm is only available for docker service update (sorry for the misleading subject that I gave to my PR)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yep, you are right.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woops, my bad. Fixed !

@sdurrheimer sdurrheimer force-pushed the zsh-completion-service-group-add-rm branch from 759d536 to 1df220d Compare September 13, 2016 08:15
@vdemeester
Copy link
Member

LGTM 🐸
Thanks @sdurrheimer @albers

@vdemeester vdemeester merged commit 0deae9f into moby:master Sep 13, 2016
@sdurrheimer sdurrheimer deleted the zsh-completion-service-group-add-rm branch September 13, 2016 08:30
@thaJeztah thaJeztah added this to the 1.13.0 milestone Sep 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants