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

Update Go versions for travis #41

Merged
merged 2 commits into from
Mar 9, 2018

Conversation

estesp
Copy link
Member

@estesp estesp commented Mar 8, 2018

Keep Go versions in line with main containerd project

Signed-off-by: Phil Estes [email protected]

@estesp estesp force-pushed the update-travis-go branch from c647669 to 7d3548a Compare March 8, 2018 20:39
@codecov-io
Copy link

codecov-io commented Mar 8, 2018

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   13.12%   13.12%           
=======================================
  Files          23       23           
  Lines        4494     4494           
=======================================
  Hits          590      590           
  Misses       3792     3792           
  Partials      112      112
Impacted Files Coverage Δ
devices.go 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf7d89f...0a357bb. Read the comment docs.

estesp added 2 commits March 8, 2018 15:51
Not sure why prior Go versions allowed this, but Go 1.10 caught the
Sprintf type mismatches.

Signed-off-by: Phil Estes <[email protected]>
@estesp estesp force-pushed the update-travis-go branch from 7d3548a to 0a357bb Compare March 8, 2018 20:51
@crosbymichael
Copy link
Member

LGTM

@crosbymichael crosbymichael merged commit c755602 into containerd:master Mar 9, 2018
@estesp estesp deleted the update-travis-go branch March 9, 2018 15:48
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.

3 participants