Skip to content
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

e2e: remove docker engine testing remnants #1993

Merged
merged 1 commit into from
Aug 15, 2019

Conversation

thaJeztah
Copy link
Member

These changes were made as part of the docker engine feature in commit fd2f1b3 (#1260), but later reverted in f250152(#1381) and b7ec4a4 (#1385)

These lines were forgotten to be removed, and should no longer be needed.

@codecov-io
Copy link

codecov-io commented Jul 10, 2019

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1993   +/-   ##
=======================================
  Coverage   56.78%   56.78%           
=======================================
  Files         311      311           
  Lines       21832    21832           
=======================================
  Hits        12398    12398           
  Misses       8519     8519           
  Partials      915      915

@thaJeztah
Copy link
Member Author

rebased to trigger CI

These changes were made as part of the `docker engine` feature
in commit fd2f1b3, but later
reverted in f250152 and
b7ec4a4

These lines were forgotten to be removed, and should no longer
be needed.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member Author

all green; bringing this one in

@thaJeztah thaJeztah merged commit aa097cf into docker:master Aug 15, 2019
@GordonTheTurtle GordonTheTurtle added this to the 19.09.0 milestone Aug 15, 2019
@thaJeztah thaJeztah deleted the e2e_remove_dead_code branch August 15, 2019 01:01
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.

4 participants