add nolints

Author: silverwind

models/migrations/v1_13/v140.go

@@ -23,7 +23,7 @@ func FixLanguageStatsToSaveSize(x *xorm.Engine) error {
 
 	const (
 		// RepoIndexerTypeCode code indexer
-		RepoIndexerTypeCode RepoIndexerType = iota // 0
+		RepoIndexerTypeCode RepoIndexerType = iota // 0 //nolint:unused
 		// RepoIndexerTypeStats repository stats indexer
 		RepoIndexerTypeStats // 1
 	)

models/migrations/v1_14/v157.go

@@ -8,13 +8,13 @@ import (
 )
 
 func FixRepoTopics(x *xorm.Engine) error {
-	type Topic struct {
+	type Topic struct { //nolint:unused
 		ID        int64  `xorm:"pk autoincr"`
 		Name      string `xorm:"UNIQUE VARCHAR(25)"`
 		RepoCount int
 	}
 
-	type RepoTopic struct {
+	type RepoTopic struct { //nolint:unused
 		RepoID  int64 `xorm:"pk"`
 		TopicID int64 `xorm:"pk"`
 	}

models/migrations/v1_14/v165.go

@@ -16,7 +16,7 @@ func ConvertHookTaskTypeToVarcharAndTrim(x *xorm.Engine) error {
 		return nil
 	}
 
-	type HookTask struct {
+	type HookTask struct { //nolint:unused
 		Typ string `xorm:"VARCHAR(16) index"`
 	}
 
@@ -42,7 +42,7 @@ func ConvertHookTaskTypeToVarcharAndTrim(x *xorm.Engine) error {
 		return err
 	}
 
-	type Webhook struct {
+	type Webhook struct { //nolint:unused
 		Type string `xorm:"VARCHAR(16) index"`
 	}
 

modules/log/event.go

@@ -82,7 +82,7 @@ func (l *ChannelledLog) Start() {
 				l.closeLogger()
 				return
 			}
-			l.loggerProvider.LogEvent(event)
+			l.loggerProvider.LogEvent(event) //nolint:errcheck
 		case _, ok := <-l.flush:
 			if !ok {
 				l.closeLogger()
@@ -119,7 +119,7 @@ func (l *ChannelledLog) emptyQueue() bool {
 			if !ok {
 				return false
 			}
-			l.loggerProvider.LogEvent(event)
+			l.loggerProvider.LogEvent(event) //nolint:errcheck
 		default:
 			return true
 		}

modules/log/file.go

@@ -181,7 +181,7 @@ func (log *FileLogger) DoRotate() error {
 		}
 
 		if log.Compress {
-			go compressOldLogFile(fname, log.CompressionLevel)
+			go compressOldLogFile(fname, log.CompressionLevel) //nolint:errcheck
 		}
 
 		// re-start logger
@@ -216,7 +216,7 @@ func compressOldLogFile(fname string, compressionLevel int) error {
 	if err != nil {
 		zw.Close()
 		fw.Close()
-		util.Remove(fname + ".gz")
+		util.Remove(fname + ".gz") //nolint:errcheck
 		return err
 	}
 	reader.Close()

modules/log/log.go

@@ -245,7 +245,7 @@ func Close() {
 func Log(skip int, level Level, format string, v ...interface{}) {
 	l, ok := NamedLoggers.Load(DEFAULT)
 	if ok {
-		l.Log(skip+1, level, format, v...)
+		l.Log(skip+1, level, format, v...) //nolint:errcheck
 	}
 }
 
@@ -273,7 +273,7 @@ func (l *LoggerAsWriter) Write(p []byte) (int, error) {
 	for _, logger := range l.ourLoggers {
 		// Skip = 3 because this presumes that we have been called by log.Println()
 		// If the caller has used log.Output or the like this will be wrong
-		logger.Log(3, l.level, string(p))
+		logger.Log(3, l.level, string(p)) //nolint:errcheck
 	}
 	return len(p), nil
 }

modules/log/logger.go

@@ -53,7 +53,7 @@ type LevelLoggerLogger struct {
 
 // Trace records trace log
 func (l *LevelLoggerLogger) Trace(format string, v ...interface{}) {
-	l.Log(1, TRACE, format, v...)
+	l.Log(1, TRACE, format, v...) //nolint:errcheck
 }
 
 // IsTrace returns true if the logger is TRACE
@@ -63,7 +63,7 @@ func (l *LevelLoggerLogger) IsTrace() bool {
 
 // Debug records debug log
 func (l *LevelLoggerLogger) Debug(format string, v ...interface{}) {
-	l.Log(1, DEBUG, format, v...)
+	l.Log(1, DEBUG, format, v...) //nolint:errcheck
 }
 
 // IsDebug returns true if the logger is DEBUG
@@ -73,7 +73,7 @@ func (l *LevelLoggerLogger) IsDebug() bool {
 
 // Info records information log
 func (l *LevelLoggerLogger) Info(format string, v ...interface{}) {
-	l.Log(1, INFO, format, v...)
+	l.Log(1, INFO, format, v...) //nolint:errcheck
 }
 
 // IsInfo returns true if the logger is INFO
@@ -83,7 +83,7 @@ func (l *LevelLoggerLogger) IsInfo() bool {
 
 // Warn records warning log
 func (l *LevelLoggerLogger) Warn(format string, v ...interface{}) {
-	l.Log(1, WARN, format, v...)
+	l.Log(1, WARN, format, v...) //nolint:errcheck
 }
 
 // IsWarn returns true if the logger is WARN
@@ -93,12 +93,12 @@ func (l *LevelLoggerLogger) IsWarn() bool {
 
 // Error records error log
 func (l *LevelLoggerLogger) Error(format string, v ...interface{}) {
-	l.Log(1, ERROR, format, v...)
+	l.Log(1, ERROR, format, v...) //nolint:errcheck
 }
 
 // ErrorWithSkip records error log from "skip" calls back from this function
 func (l *LevelLoggerLogger) ErrorWithSkip(skip int, format string, v ...interface{}) {
-	l.Log(skip+1, ERROR, format, v...)
+	l.Log(skip+1, ERROR, format, v...) //nolint:errcheck
 }
 
 // IsError returns true if the logger is ERROR
@@ -108,12 +108,12 @@ func (l *LevelLoggerLogger) IsError() bool {
 
 // Critical records critical log
 func (l *LevelLoggerLogger) Critical(format string, v ...interface{}) {
-	l.Log(1, CRITICAL, format, v...)
+	l.Log(1, CRITICAL, format, v...) //nolint:errcheck
 }
 
 // CriticalWithSkip records critical log from "skip" calls back from this function
 func (l *LevelLoggerLogger) CriticalWithSkip(skip int, format string, v ...interface{}) {
-	l.Log(skip+1, CRITICAL, format, v...)
+	l.Log(skip+1, CRITICAL, format, v...) //nolint:errcheck
 }
 
 // IsCritical returns true if the logger is CRITICAL
@@ -123,14 +123,14 @@ func (l *LevelLoggerLogger) IsCritical() bool {
 
 // Fatal records fatal log and exit the process
 func (l *LevelLoggerLogger) Fatal(format string, v ...interface{}) {
-	l.Log(1, FATAL, format, v...)
+	l.Log(1, FATAL, format, v...) //nolint:errcheck
 	l.Close()
 	os.Exit(1)
 }
 
 // FatalWithSkip records fatal log from "skip" calls back from this function and exits the process
 func (l *LevelLoggerLogger) FatalWithSkip(skip int, format string, v ...interface{}) {
-	l.Log(skip+1, FATAL, format, v...)
+	l.Log(skip+1, FATAL, format, v...) //nolint:errcheck
 	l.Close()
 	os.Exit(1)
 }

modules/log/multichannel.go

@@ -99,6 +99,6 @@ func (l *MultiChannelledLogger) SendLog(level Level, caller, filename string, li
 		time:       time.Now(),
 		stacktrace: stack,
 	}
-	l.LogEvent(event)
+	l.LogEvent(event) //nolint:errcheck
 	return nil
 }

modules/log/writer.go

@@ -202,7 +202,7 @@ func (logger *WriterLogger) createMsg(buf *[]byte, event *Event) {
 	(&protectedANSIWriter{
 		w:    &baw,
 		mode: pawMode,
-	}).Write(msg)
+	}).Write(msg) //nolint:errcheck
 	*buf = baw
 
 	if event.stacktrace != "" && logger.StacktraceLevel <= event.level {
@@ -249,7 +249,7 @@ func (logger *WriterLogger) Match(event *Event) bool {
 	(&protectedANSIWriter{
 		w:    &baw,
 		mode: removeColor,
-	}).Write([]byte(event.msg))
+	}).Write([]byte(event.msg)) //nolint:errcheck
 	msg = baw
 	return logger.regexp.Match(msg)
 }

modules/user/user.go

@@ -16,7 +16,7 @@ func CurrentUsername() string {
 	if err != nil {
 		return fallbackCurrentUsername()
 	}
-	username := userinfo.Username
+	username := userinfo.Username //nolint:revive
 	if runtime.GOOS == "windows" {
 		parts := strings.Split(username, "\\")
 		username = parts[len(parts)-1]