Skip to content
New issue

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

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

Already on GitHub? # to your account

Sort docker stack ls by name #31085

Merged
merged 1 commit into from
Feb 16, 2017
Merged

Sort docker stack ls by name #31085

merged 1 commit into from
Feb 16, 2017

Conversation

vdemeester
Copy link
Member

Part of #30977 — let's sort the stack by name. At some point we might want to add --format and --sort-by to this command, but could be in a follow-up 👼.

/cc @dnephin @thaJeztah @icecrime

🐸

Signed-off-by: Vincent Demeester vincent@sbr.pm

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Copy link
Member

@dnephin dnephin left a comment

Choose a reason for hiding this comment

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

Copy link
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM (design and code)

@dnephin dnephin merged commit 3e53eca into moby:master Feb 16, 2017
@GordonTheTurtle GordonTheTurtle added this to the 1.14.0 milestone Feb 16, 2017
@vdemeester vdemeester deleted the sort-stack-ls branch February 16, 2017 19:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants