Skip to content

Bump Golang 1.12.3 and fix: nolint directive - #1820

Closed
thaJeztah wants to merge 2 commits into
docker:masterfrom
thaJeztah:fix_lint
Closed

Bump Golang 1.12.3 and fix: nolint directive#1820
thaJeztah wants to merge 2 commits into
docker:masterfrom
thaJeztah:fix_lint

Conversation

@thaJeztah

@thaJeztahthaJeztah commented Apr 11, 2019

Copy link
Copy Markdown
Member

relates to #1818 (comment)
carries #1818
closes#1818

let's see if current go version doesn't complain without this statement 🤷‍♂️

Signed-off-by: Jintao Zhang <zhangjintao9020@gmail.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-io

Copy link
Copy Markdown

Codecov Report

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

@@ Coverage Diff @@## master #1820 +/- ##
=======================================
Coverage 56.31% 56.31% =======================================
Files 308 308 Lines 21429 21429 =======================================
Hits 12068 12068 Misses 8476 8476 Partials 885 885

@thaJeztahthaJeztah changed the title fix: nolint directive did not match any issue (nolint)Bump Golang 1.12.3 and fix: nolint directiveApr 11, 2019
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

LOL; and removing it also says it isn't right 😂

Successfully tagged cli-linter:24327
cli/command/formatter/reflect_test.go:15:17:warning: func (*dummy).func2 is unused (U1000) (unused)

Closing this one

@thaJeztah
thaJeztah deleted the fix_lint branch April 11, 2019 19:57
Sign up for freeto 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.

4 participants

@thaJeztah@codecov-io@GordonTheTurtle@tao12345666333