Skip to content

add go.mod#26

Merged
fuweid merged 1 commit intocontainerd:masterfrom
AkihiroSuda:gomod
Dec 19, 2019
Merged

add go.mod#26
fuweid merged 1 commit intocontainerd:masterfrom
AkihiroSuda:gomod

Conversation

@AkihiroSuda
Copy link
Member

@codecov-io
Copy link

codecov-io commented Dec 13, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   55.68%   55.68%           
=======================================
  Files           1        1           
  Lines         176      176           
=======================================
  Hits           98       98           
  Misses         51       51           
  Partials       27       27

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 9abf673...7b3b1a5. Read the comment docs.

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Copy link
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

This was referenced Dec 14, 2019
- cd $GOPATH/src/github.com/containerd/zfs
- go get -t ./...
- GO111MODULE="on" go mod vendor
- go get -u github.com/vbatts/git-validation
Copy link
Member

Choose a reason for hiding this comment

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

after we add vendor check, we cannot call get get in module because it will change existing file. will change it in followup.

Copy link
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@fuweid fuweid merged commit 56b3b5c into containerd:master Dec 19, 2019
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.

4 participants