Skip to content

Update API history and man page for --filter expose/publish#30661

Merged
thaJeztah merged 1 commit intomoby:masterfrom
yongtang:02012017-man-expose-publish-filter
Feb 7, 2017
Merged

Update API history and man page for --filter expose/publish#30661
thaJeztah merged 1 commit intomoby:masterfrom
yongtang:02012017-man-expose-publish-filter

Conversation

@yongtang
Copy link
Member

@yongtang yongtang commented Feb 2, 2017

This fix updates API history and man page for
docker ps --filter expose/publish, from the feedback:
#27557 (comment)

Signed-off-by: Yong Tang [email protected]

/cc @thaJeztah

Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 👼
@yongtang needs a rebase 😛
/cc @thaJeztah

@yongtang yongtang force-pushed the 02012017-man-expose-publish-filter branch from f5667b8 to c996ce4 Compare February 2, 2017 12:47
@yongtang
Copy link
Member Author

yongtang commented Feb 2, 2017

@vdemeester The PR has been rebased. Thanks.

@vdemeester
Copy link
Member

@yongtang needs another rebase
@thaJeztah needs a review 👼 🙏

This fix updates API history and man page for
`docker ps --filter expose/publish`, from the feedback:
moby#27557 (comment)

Signed-off-by: Yong Tang <[email protected]>
@yongtang yongtang force-pushed the 02012017-man-expose-publish-filter branch from c996ce4 to 9fc3936 Compare February 7, 2017 16:25
@yongtang
Copy link
Member Author

yongtang commented Feb 7, 2017

@vdemeester @thaJeztah The PR has been rebased. Thanks.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@thaJeztah thaJeztah merged commit 91f36e4 into moby:master Feb 7, 2017
@GordonTheTurtle GordonTheTurtle added this to the 1.14.0 milestone Feb 7, 2017
@yongtang yongtang deleted the 02012017-man-expose-publish-filter branch February 7, 2017 22:09
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.

5 participants