Bump golangci-lint version

.github/workflows/main.yml

@@ -5,7 +5,7 @@ on:
 env:
   GOFUMPT_VERSION: 0.6.0
   GOTOOLCHAIN: local
-  GOLANGCI_LINT_VERSION: 1.56.2
+  GOLANGCI_LINT_VERSION: 1.59.1
 jobs:
   linux:
     runs-on: ubuntu-latest

.golangci.yml

@@ -4,18 +4,20 @@ linters:
   - asciicheck
   - bidichk
   - bodyclose
+  - canonicalheader
   - containedctx
   - contextcheck
   - decorder
   - dogsled
   - dupl
   - dupword
   - durationcheck
+  - err113
   - errcheck
   - errchkjson
   - errorlint
-  - execinquery
   - exportloopref
+  - fatcontext
   - forbidigo
   - forcetypeassert
   - gci
@@ -25,7 +27,6 @@ linters:
   - gochecksumtype
   - gocritic
   - godot
-  - goerr113
   - gofmt
   - gofumpt
   - goimports
@@ -78,6 +79,7 @@ linters:
   - unparam
   - unused
   - usestdlibvars
+  - wastedassign
   - whitespace
   - zerologlint
   disable:
@@ -104,7 +106,6 @@ linters:
   - testpackage
   - testpackage
   - varnamelen
-  - wastedassign # https://github.com/golangci/golangci-lint/issues/2649
   - wrapcheck
   - wsl
 
@@ -149,7 +150,7 @@ issues:
     - godot
     path: "^xyz/"
   - linters:
-    - goerr113
+    - err113
     text: "do not define dynamic errors, use wrapped static errors instead"
   - linters:
     - forbidigo

encoding/ewkb/sql.go

@@ -90,7 +90,7 @@ func (p *Point) Valid() bool {
 // Value returns the EWKB encoding of p.
 func (p *Point) Value() (driver.Value, error) {
 	if p.Point == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(p.Point)
 }
@@ -125,7 +125,7 @@ func (ls *LineString) Valid() bool {
 // Value returns the EWKB encoding of ls.
 func (ls *LineString) Value() (driver.Value, error) {
 	if ls.LineString == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(ls.LineString)
 }
@@ -160,7 +160,7 @@ func (p *Polygon) Valid() bool {
 // Value returns the EWKB encoding of p.
 func (p *Polygon) Value() (driver.Value, error) {
 	if p.Polygon == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(p.Polygon)
 }
@@ -195,7 +195,7 @@ func (mp *MultiPoint) Valid() bool {
 // Value returns the EWKB encoding of mp.
 func (mp *MultiPoint) Value() (driver.Value, error) {
 	if mp.MultiPoint == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(mp.MultiPoint)
 }
@@ -230,7 +230,7 @@ func (mls *MultiLineString) Valid() bool {
 // Value returns the EWKB encoding of mls.
 func (mls *MultiLineString) Value() (driver.Value, error) {
 	if mls.MultiLineString == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(mls.MultiLineString)
 }
@@ -265,7 +265,7 @@ func (mp *MultiPolygon) Valid() bool {
 // Value returns the EWKB encoding of mp.
 func (mp *MultiPolygon) Value() (driver.Value, error) {
 	if mp.MultiPolygon == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(mp.MultiPolygon)
 }
@@ -300,7 +300,7 @@ func (gc *GeometryCollection) Valid() bool {
 // Value returns the EWKB encoding of gc.
 func (gc *GeometryCollection) Value() (driver.Value, error) {
 	if gc.GeometryCollection == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	return value(gc.GeometryCollection)
 }

encoding/geojson/geojson.go

@@ -114,7 +114,7 @@ func guessLayout3(coords3 [][][]geom.Coord) (geom.Layout, error) {
 // Decode decodes g to a geometry.
 func (g *Geometry) Decode() (geom.T, error) {
 	if g == nil {
-		return nil, nil
+		return nil, nil //nolint:nilnil
 	}
 	switch g.Type {
 	case "Point":