Lint: Add linter for log statements #3663
Conversation
Adds linter for log statements. Also, fix all the places the linter flagged in the codebase.
There was a problem hiding this comment.
Reviewed 5 of 5 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @oncilla)
a discussion (no related file):
hm I just realized that the case in the BR is not catched: https://github.com/scionproto/scion/blob/master/go/border/rctrl/ifstate.go#L45
But it's certainly better than nothing.
There was a problem hiding this comment.
Reviewed 1 of 1 files at r2.
Reviewable status:complete! all files reviewed, all discussions resolved
|
a discussion (no related file): Previously, lukedirtwalker (Lukas Vogel) wrote…
created oncilla/gochecks#6 to track this |
Adds linter for log statements. Also, fix all the places the linter flagged in the codebase.
Adds linter for log statements.
Also, fix all the places the linter flagged in the codebase.
This change is