Skip to content

Update files based on go lint#1577

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
crosbymichael:lint-1
Oct 2, 2017
Merged

Update files based on go lint#1577
crosbymichael merged 1 commit intocontainerd:masterfrom
crosbymichael:lint-1

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

Continuing work on making the code base go lint approved.

Signed-off-by: Michael Crosby [email protected]

@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 29, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1577   +/-   ##
=======================================
  Coverage   42.36%   42.36%           
=======================================
  Files          24       24           
  Lines        3368     3368           
=======================================
  Hits         1427     1427           
  Misses       1612     1612           
  Partials      329      329
Impacted Files Coverage Δ
fs/du.go 0% <ø> (ø) ⬆️
fs/hardlink.go 83.33% <ø> (ø) ⬆️

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 579093f...f43b7ac. Read the comment docs.

Comment thread content/content.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

// ReaderAt extends the standard io.ReaderAt interface with reporting of Size and io.Closer.

@stevvooe
Copy link
Copy Markdown
Member

LGTM

I have a small suggestion but we can work to make these godocs better as we go.

Comment thread fs/du.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nit: returns -> is

Signed-off-by: Michael Crosby <[email protected]>
Copy link
Copy Markdown
Contributor

@mlaventure mlaventure left a comment

Choose a reason for hiding this comment

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

LGTM

@crosbymichael crosbymichael merged commit 33e974c into containerd:master Oct 2, 2017
@crosbymichael crosbymichael deleted the lint-1 branch October 2, 2017 14:57
mauriciovasquezbernal pushed a commit to kinvolk/containerd that referenced this pull request Nov 13, 2020
Make script work for both python2.7 and 3.6
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