Skip to content

Update logrus to v1.0.1 - with networkd, runc, containerd masters#2345

Merged
cyli merged 4 commits intomoby:masterfrom
simonferquel:update-logrus
Aug 18, 2017
Merged

Update logrus to v1.0.1 - with networkd, runc, containerd masters#2345
cyli merged 4 commits intomoby:masterfrom
simonferquel:update-logrus

Conversation

@simonferquel
Copy link
Contributor

This is based on #2334 (thanks @dmcgowan), and implements @ijc instructions.
Not sure if the vendored version are completely correct though (I picked masters current heads)

dmcgowan and others added 4 commits August 9, 2017 16:08
Fixes case sensitivity issue

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
Signed-off-by: Derek McGowan <derek@mcgstyle.net>
Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
@codecov
Copy link

codecov bot commented Aug 9, 2017

Codecov Report

Merging #2345 into master will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2345      +/-   ##
==========================================
+ Coverage   60.15%   60.36%   +0.21%     
==========================================
  Files         128      128              
  Lines       26052    26052              
==========================================
+ Hits        15671    15726      +55     
+ Misses       8982     8936      -46     
+ Partials     1399     1390       -9

@ijc
Copy link
Contributor

ijc commented Aug 14, 2017

The containerd changed LGTM

@thaJeztah
Copy link
Member

ping @cyli @dmcgowan PTAL

@dmcgowan
Copy link
Member

Updated versions look good to me, I suggest moving forward with this to avoid making it an even bigger headache

@cyli
Copy link
Contributor

cyli commented Aug 18, 2017

LGTM

@cyli cyli merged commit ddb4539 into moby:master Aug 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants