diff --git a/.golangci.yaml b/.golangci.yaml index a6260b8..e516efd 100644 --- a/.golangci.yaml +++ b/.golangci.yaml @@ -19,11 +19,10 @@ linters: - noctx - funlen - nestif + - gocognit - interfacer - revive - cyclop - - goerr113 - - gocognit # deprecated: - scopelint diff --git a/internal/types/errors.go b/internal/types/errors.go index 89d438d..62c36f1 100644 --- a/internal/types/errors.go +++ b/internal/types/errors.go @@ -6,4 +6,7 @@ var ( ErrNotFound = errors.New("cache entry not found") ErrInvalidCacheEntry = errors.New("invalid cache entry format") ErrUpstream = errors.New("upstream error") + + // ErrNoServersConfigured means that there are no servers to run. + ErrNoServersConfigured = errors.New("no servers configured") ) diff --git a/srv.go b/srv.go index 22b26d7..8f4e88d 100644 --- a/srv.go +++ b/srv.go @@ -2,7 +2,6 @@ package main import ( "crypto/tls" - "errors" "fmt" "io" "log" @@ -16,6 +15,7 @@ import ( geoloc "github.com/chubin/wttr.in/internal/geo/location" "github.com/chubin/wttr.in/internal/logging" "github.com/chubin/wttr.in/internal/processor" + "github.com/chubin/wttr.in/internal/types" ) //nolint:gochecknoglobals @@ -158,7 +158,7 @@ func serve(conf *config.Config) error { numberOfServers++ } if numberOfServers == 0 { - return errors.New("no servers configured") + return types.ErrNoServersConfigured } return <-errs // block until one of the servers writes an error