-
Notifications
You must be signed in to change notification settings - Fork 2.1k
introduce context show command
#3567
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
Conversation
thaJeztah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments.
Now that we have the docker context show command, perhaps we should
- add a
docs/reference/context_show.mdfile - move the examples from https://github.com/docker/cli/pull/2500/files#diff-3589954942dcb3aa5aa11ca30bee4e904ebd9c35d1d9dfa73192f1655cdc5250R81
- (or keep the heading that's there, and link to the
context showreference)
Codecov Report
@@ Coverage Diff @@
## master #3567 +/- ##
==========================================
- Coverage 58.99% 58.98% -0.02%
==========================================
Files 282 285 +3
Lines 23813 23834 +21
==========================================
+ Hits 14049 14058 +9
- Misses 8908 8920 +12
Partials 856 856 |
c6b1518 to
e7fe22f
Compare
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
thaJeztah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
closes #3425
- What I did
introduce
context showcommand, aligned with same command introduce in compose-cli- How I did it
new cobra command, dump current context name after checking it exists
- How to verify it
docker context show- Description for the changelog
introduce
docker context showcommand- A picture of a cute animal (not mandatory but encouraged)
