Skip to content

Commit b3e9721

Browse files
committed
Add issue reference to nolint annotation
Usually errorlint allows io.EOF comparison (based on a whitelist of functions that can return bare io.EOF), thus there is no need for nolint annotation. In this very case, though, the need for nolint is caused by issue with errorlint, which fails to see where err is coming from. Refer to the issue so when it is fixed we can remove the annotation. Signed-off-by: Kir Kolyshkin <[email protected]>
1 parent 693d1c6 commit b3e9721

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

notify_socket.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ func sdNotifyBarrier(client *net.UnixConn) error {
215215
// Probably the other end doesn't support the sd_notify_barrier protocol.
216216
logrus.Warn("Timeout after waiting 30s for barrier. Ignored.")
217217
return nil
218-
} else if err == io.EOF { //nolint:errorlint // comparison with io.EOF is legit.
218+
} else if err == io.EOF { //nolint:errorlint // https://github.com/polyfloyd/go-errorlint/issues/49
219219
return nil
220220
} else {
221221
return err

0 commit comments

Comments
 (0)