diff options
| author | Alexander Mazuruk <a.mazuruk@samsung.com> | 2018-11-23 16:32:43 +0100 |
|---|---|---|
| committer | Alexander Mazuruk <a.mazuruk@samsung.com> | 2018-12-11 17:59:40 +0100 |
| commit | 2004de3499eb1904ab51c83c85c7d14178a8531a (patch) | |
| tree | 7c5eabae4b77d6341bb9b612db287501093feebe /controller/jobscontrollerimpl.go | |
| parent | 7268cf53587a7fe5269ec01c037c9666e419c15b (diff) | |
| download | weles-2004de3499eb1904ab51c83c85c7d14178a8531a.tar.gz weles-2004de3499eb1904ab51c83c85c7d14178a8531a.tar.bz2 weles-2004de3499eb1904ab51c83c85c7d14178a8531a.zip | |
Fix linter errors
Few errors introduced while CI was off, few due to updated checkers.
All fixed in this commit.
Command line help message was adjusted.
Additionally unified "// nolint:linter" comments - one space between text
and "//", rest made as short as possible (no spaces)
Verification:
$ gometalinter --deadline=60s --config .gometalinter.json ./...
Change-Id: Ibc18c019385bfb7207b4375d19cb5812bbbf987a
Signed-off-by: Alexander Mazuruk <a.mazuruk@samsung.com>
Diffstat (limited to 'controller/jobscontrollerimpl.go')
| -rw-r--r-- | controller/jobscontrollerimpl.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/controller/jobscontrollerimpl.go b/controller/jobscontrollerimpl.go index 78f4cdd..dfbdff7 100644 --- a/controller/jobscontrollerimpl.go +++ b/controller/jobscontrollerimpl.go @@ -396,7 +396,8 @@ func prepareFilterRegexp(arr []string) (*regexp.Regexp, error) { var str strings.Builder str.Grow(size) for _, s := range arr { - _, _ = str.WriteString("|(" + s + ")") + // WriteString never returns error (as of go 1.11.2) thus err is ignored. + _, _ = str.WriteString("|(" + s + ")") // nolint:gosec } return regexp.Compile(str.String()[1:]) |