Skip to content

Fix it rebase #631

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.

Sign up for GitHub

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

Jump to bottom
Merged
KostyaSha merged 8 commits into from
Jul 8, 2016
Merged

Fix it rebase #631

KostyaSha merged 8 commits into from
Jul 8, 2016

Conversation

@KostyaSha
Copy link
Member

@KostyaSha KostyaSha commented Jul 7, 2016 •
edited by marcuslinke
Loading

#620 rebase

This change is Reviewable

KostyaSha added 5 commits July 8, 2016 00:22
@KostyaSha
Update deps. Fix IT.
fba9639 
- remove custom jersey connector
- update all deps
- httpclient latest fails, stay on working version
@KostyaSha
revert dep and test
52ff6d8
@KostyaSha
Skip auth tests
1a148f2
@KostyaSha
Enable cache
3a782a9
@KostyaSha
Disable experimental/testing
31d5627
@codecov-io
Copy link

codecov-io commented Jul 7, 2016 •
edited
Loading

Current coverage is 70.29%

Merging #631 into master will increase coverage by 46.84%

@@             master       #631   diff @@
==========================================
  Files           296        293      -3   
  Lines          6257       6026    -231   
  Methods           0          0           
  Messages          0          0           
  Branches        560        511     -49   
==========================================
+ Hits           1467       4236   +2769   
+ Misses         4693       1558   -3135   
- Partials         97        232    +135

Powered by Codecov. Last updated by e5a1cd8...3a782a9

@KostyaSha
Copy link
Member Author

KostyaSha commented Jul 7, 2016

Current coverage is 70.29%

Nice bug... i will switch coverage to some successful IT branch.

@KostyaSha
Copy link
Member Author

KostyaSha commented Jul 8, 2016

The same tests fails with unix socket as current master, so merging

Failed tests: 
  CopyArchiveToContainerCmdImplTest.copyFileToContainer:57->assertFileCopied:83 » Processing
  CopyArchiveToContainerCmdImplTest.copyStreamToContainer:66->assertFileCopied:83 » Processing
  EventsCmdImplTest.testEventStreaming1:87 » Processing Error closing message co...

@KostyaSha KostyaSha merged commit d53eef0 into docker-java:master Jul 8, 2016
panuse pushed a commit to TuKangTech/docker-java that referenced this pull request Aug 20, 2017
@KostyaSha
Merge pull request docker-java#631 from KostyaSha/fixIT-rebase
106a2bd 
Fix it rebase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KostyaSha @codecov-io