Skip to content

Try set GOGC for golint#3657

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
crosbymichael:golint
Sep 17, 2019
Merged

Try set GOGC for golint#3657
dmcgowan merged 1 commit intocontainerd:masterfrom
crosbymichael:golint

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

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

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

@crosbymichael crosbymichael force-pushed the golint branch 2 times, most recently from 7ffb10b to 603884d Compare September 17, 2019 17:29
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

@crosbymichael crosbymichael force-pushed the golint branch 2 times, most recently from 6547760 to 26ea074 Compare September 17, 2019 17:57
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

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

theopenlab-ci Bot commented Sep 17, 2019

Build succeeded.

Copy link
Copy Markdown
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

Looks like GOGC=75 + v1.18 of golangci-lint is a working combination for both Travis and Appveyor

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3657   +/-   ##
=======================================
  Coverage   42.39%   42.39%           
=======================================
  Files         127      127           
  Lines       14048    14048           
=======================================
  Hits         5955     5955           
  Misses       7193     7193           
  Partials      900      900
Flag Coverage Δ
#linux 45.9% <ø> (ø) ⬆️
#windows 37.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 86442df...3bc9975. Read the comment docs.

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

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