Skip to content

Conversation

@thaJeztah
Copy link
Member

This makes running the client easier inside
the container; allowing to use just docker
instead of build/docker.

- What I did

- How I did it

- How to verify it

$ make -f docker.Makefile binary shell
$ docker version

This makes running the client easier inside
the container; allowing to use just `docker`
instead of `build/docker`.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-io
Copy link

codecov-io commented Aug 5, 2017

Codecov Report

Merging #419 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #419   +/-   ##
=======================================
  Coverage   46.25%   46.25%           
=======================================
  Files         193      193           
  Lines       16097    16097           
=======================================
  Hits         7445     7445           
  Misses       8263     8263           
  Partials      389      389

@thaJeztah
Copy link
Member Author

ping @vdemeester 😄

Copy link
Collaborator

@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 🐸
/cc @dnephin

Copy link
Contributor

@mlaventure mlaventure left a comment

Choose a reason for hiding this comment

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

LGTM

@mlaventure mlaventure merged commit e0f1e4d into docker:master Aug 7, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.08.0 milestone Aug 7, 2017
@thaJeztah thaJeztah deleted the easier-testing-in-container branch August 7, 2017 16:19
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