diff options
author | Pavlo Strokov <pstrokov@gitlab.com> | 2021-08-04 15:41:24 +0300 |
---|---|---|
committer | Pavlo Strokov <pstrokov@gitlab.com> | 2021-08-09 10:59:36 +0300 |
commit | 631638faef0292271e674cefeb87cf3af60f4281 (patch) | |
tree | 2c11a235f1526d4222c63211f95194073948609c /.golangci.yml | |
parent | 5954eb119126a97c80a0d6e69114b8e1bc349b7e (diff) |
lint: Handle error returned by os.RemoveAll
The change removes os.RemoveAll from lint rules exclusion
and fixes all the places where it causes the issue.
If removal ends up with an error we try to log it if the
logger is accessible. Otherwise the error is just omitted.
We don't return it back to the caller because we don't want
functional changes here.
Diffstat (limited to '.golangci.yml')
-rw-r--r-- | .golangci.yml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/.golangci.yml b/.golangci.yml index e499c84a5..4a0c9353b 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -573,7 +573,7 @@ issues: text: "Error return value of `[^`]+.(Close|Serve)` is not checked" - linters: - errcheck - text: "Error return value of `(out\\.Flush|os\\.Remove(All)?)` is not checked" + text: "Error return value of `(out\\.Flush|os\\.Remove)` is not checked" ## errcheck: Specific issues in non-*_test.go files - linters: - errcheck |