Skip to content

Sync binary commits with vndr#36335

Merged
vdemeester merged 3 commits intomoby:masterfrom
thaJeztah:sync-libnetwork-commits
Feb 16, 2018
Merged

Sync binary commits with vndr#36335
vdemeester merged 3 commits intomoby:masterfrom
thaJeztah:sync-libnetwork-commits

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Mainly for "hygiene";

ping @vdemeester @AkihiroSuda PTAL

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
The BurntSushi/toml code is now re-licensed as MIT. While
the vendored package was already updated, the tomlv binary
used was still using the old license type.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Copy link
Copy Markdown
Member

@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 🐯

Copy link
Copy Markdown
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

LGTM BTW do we plan to migrate to go dep? (after go 1.1x?)

@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda I think @tiborvass was looking at that as an option

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 16, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@04d9726). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #36335   +/-   ##
=========================================
  Coverage          ?   34.22%           
=========================================
  Files             ?      609           
  Lines             ?    45267           
  Branches          ?        0           
=========================================
  Hits              ?    15493           
  Misses            ?    27821           
  Partials          ?     1953

@vdemeester vdemeester merged commit 302e584 into moby:master Feb 16, 2018
@thaJeztah thaJeztah deleted the sync-libnetwork-commits branch February 16, 2018 15:23
@dnephin
Copy link
Copy Markdown
Member

dnephin commented Feb 16, 2018

BTW do we plan to migrate to go dep?

I think we should

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