Skip to content
This repository has been archived by the owner on Jun 13, 2021. It is now read-only.

App inspect requires exactly one arg #682

Merged
merged 1 commit into from Oct 11, 2019
Merged

Conversation

rumpl
Copy link
Member

@rumpl rumpl commented Oct 11, 2019

image

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

@rumpl rumpl force-pushed the inspect-exact-args branch 2 times, most recently from 182ae64 to 8767db0 Compare October 11, 2019 10:27
@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@3b9a6e3). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #682   +/-   ##
=========================================
  Coverage          ?   71.77%           
=========================================
  Files             ?       56           
  Lines             ?     2888           
  Branches          ?        0           
=========================================
  Hits              ?     2073           
  Misses            ?      551           
  Partials          ?      264
Impacted Files Coverage Δ
internal/commands/inspect.go 69.23% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b9a6e3...252c051. Read the comment docs.

Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl rumpl merged commit cf8ec17 into docker:master Oct 11, 2019
@rumpl rumpl deleted the inspect-exact-args branch October 18, 2019 08:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants