Upgrade golangci-lint to v1.55.2

Some of the changes involve the `revive` linter being a bit stricter,
resulting in the removal of "dot" imports. This is probably a good
thing, since the tests are now more similar to how users will see
things. There is also a new linter used called `perfsprint` which checks
for bad uses of `fmt.Sprintf`.

.ci/golint.Dockerfile

@@ -1,4 +1,4 @@
-FROM golangci/golangci-lint:v1.54.2
+FROM golangci/golangci-lint:v1.55.2
 
 RUN apt-get update \
   && apt-get install -y -q --no-install-recommends \

.golangci.yaml

@@ -83,6 +83,7 @@ linters:
     - noctx
     - nolintlint
     - nosprintfhostport
+    - perfsprint
     - predeclared
     - promlinter
     - reassign
@@ -133,19 +134,24 @@ linters:
     - gci
     - gochecknoglobals
     - gochecknoinits
+    - gochecksumtype
     - gocognit
     - goconst
     - gocyclo
     - godox
     - goerr113
     - gomnd
+    - inamedparam
     - lll
     - maintidx
     - nestif
     - nlreturn
     - nonamedreturns
     - paralleltest
     - prealloc
+    - protogetter
+    - sloglint
+    - testifylint
     - varnamelen
     - wrapcheck
     - wsl

Makefile

@@ -3,7 +3,8 @@ DC_RUN = \
 	--project-name sf-$$task \
 	--file .ci/docker-compose-$$task.yml \
 	up \
-	--abort-on-container-exit
+	--abort-on-container-exit \
+	--build
 
 .PHONY: lint
 lint:

geom/accessor_test.go

@@ -3,13 +3,13 @@ package geom_test
 import (
 	"testing"
 
-	. "github.com/peterstace/simplefeatures/geom"
+	"github.com/peterstace/simplefeatures/geom"
 )
 
 func TestPointAccessorNonEmpty(t *testing.T) {
 	xy, ok := geomFromWKT(t, "POINT(1 2)").MustAsPoint().XY()
 	expectBoolEq(t, ok, true)
-	expectXYEq(t, xy, XY{1, 2})
+	expectXYEq(t, xy, geom.XY{1, 2})
 }
 
 func TestPointAccessorEmpty(t *testing.T) {
@@ -25,11 +25,11 @@ func TestLineStringAccessor(t *testing.T) {
 	pt56 := xyCoords(5, 6)
 
 	t.Run("start", func(t *testing.T) {
-		want := NewPoint(pt12)
+		want := geom.NewPoint(pt12)
 		expectGeomEq(t, ls.StartPoint().AsGeometry(), want.AsGeometry())
 	})
 	t.Run("end", func(t *testing.T) {
-		want := NewPoint(pt56)
+		want := geom.NewPoint(pt56)
 		expectGeomEq(t, ls.EndPoint().AsGeometry(), want.AsGeometry())
 	})
 	t.Run("num points", func(t *testing.T) {