chore: update build infrastructure

.github/workflows/main.yml

@@ -10,10 +10,18 @@ on:
     - v*
 jobs:
   test:
+    strategy:
+      fail-fast: false
+      matrix:
+        go-version:
+        - stable
+        - oldstable
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@8e5e7e5ab8b370d6c329ec480221332ada57f0ab
     - uses: actions/setup-go@fac708d6674e30b6ba41289acaab6d4b75aa0753
+      with:
+        go-version: ${{ matrix.go-version }}
     - name: build
       run: go build ./...
     - name: test
@@ -25,4 +33,4 @@ jobs:
     - uses: actions/setup-go@fac708d6674e30b6ba41289acaab6d4b75aa0753
     - uses: golangci/golangci-lint-action@08e2f20817b15149a52b5b3ebe7de50aff2ba8c5
       with:
-        version: v1.53.2
+        version: v1.54.2

.golangci.yml

@@ -1,36 +1,85 @@
 linters:
   enable:
+  - asasalint
   - asciicheck
+  - bidichk
   - bodyclose
+  - containedctx
+  - contextcheck
+  - decorder
   - dogsled
   - dupl
+  - dupword
+  - durationcheck
   - errcheck
+  - errchkjson
+  - errname
+  - errorlint
+  - execinquery
   - exhaustive
   - exportloopref
+  - forbidigo
+  - forcetypeassert
+  - gci
+  - ginkgolinter
+  - gocheckcompilerdirectives
+  - gochecknoinits
   - goconst
   - gocritic
   - godot
   - gofmt
   - gofumpt
+  - goheader
   - goimports
+  - gomoddirectives
   - gomodguard
   - goprintffuncname
   - gosimple
+  - gosmopolitan
   - govet
+  - grouper
+  - importas
   - ineffassign
+  - interfacebloat
+  - ireturn
+  - loggercheck
+  - makezero
+  - mirror
   - misspell
+  - musttag
   - nakedret
+  - nilerr
+  - nilnil
   - noctx
   - nolintlint
+  - nonamedreturns
+  - nosprintfhostport
+  - paralleltest
   - prealloc
+  - predeclared
+  - promlinter
+  - reassign
+  - rowserrcheck
+  - sqlclosecheck
   - staticcheck
+  - tagalign
+  - tagliatelle
+  - tenv
+  - testableexamples
+  - thelper
+  - tparallel
   - typecheck
   - unconvert
   - unparam
   - unused
+  - usestdlibvars
+  - wastedassign
   - whitespace
+  - zerologlint
   disable:
+  - cyclop
   - depguard
+  - exhaustruct
   - funlen
   - gochecknoglobals
   - gocognit
@@ -40,14 +89,26 @@ linters:
   - gomnd
   - gosec
   - lll
+  - maintidx
   - maligned
   - nestif
-  - sqlclosecheck
+  - nlreturn
+  - revive
   - stylecheck
   - testpackage
+  - varnamelen
+  - wrapcheck
   - wsl
 
 linters-settings:
+  gci:
+    sections:
+    - standard
+    - default
+    - prefix(github.com/twpayne/go-xmlstruct)
+  gofumpt:
+    extra-rules: true
+    module-path: github.com/twpayne/go-xmlstruct
   goimports:
     local-prefixes: github.com/twpayne/go-xmlstruct
 
@@ -57,3 +118,6 @@ issues:
     - dupl
     - scopelint
     path: "_test\\.go"
+  - linters:
+    - forbidigo
+    path: "main\\.go"

README.md

@@ -16,7 +16,7 @@ code](https://github.com/twpayne/go-xmlstruct/blob/master/internal/tests/play/pl
 Compared to existing Go struct generators like
 [zek](https://github.com/miku/zek),
 [XMLGen](https://github.com/dutchcoders/XMLGen), and
-[chidley](https://github.com/gnewton/chidley), go-xmlstruct offers:
+[chidley](https://github.com/gnewton/chidley), go-xmlstruct:
 
 * Takes multiple XML documents as input.
 * Generates field types of `bool`, `int`, `string`, or `time.Time` as

generator.go

@@ -19,7 +19,7 @@ import (
 var (
 	SkipDir = fs.SkipDir
 	//lint:ignore ST1012 SkipFile is not an error
-	SkipFile = errors.New("skip file")
+	SkipFile = errors.New("skip file") //nolint:errname
 )
 
 // A Generator observes XML documents and generates Go structs into which the

generator_test.go

@@ -10,6 +10,8 @@ import (
 )
 
 func TestGenerator(t *testing.T) {
+	t.Parallel()
+
 	for _, tc := range []struct {
 		name        string
 		xmlStr      string
@@ -222,7 +224,7 @@ func TestGenerator(t *testing.T) {
 				`package main`,
 				``,
 				`type A struct {`,
-				"\tB B `xml:\"b\"`",
+				"\tB B `xml:\"b\"`", //nolint:dupword
 				`}`,
 				``,
 				`type B struct {`,
@@ -317,7 +319,10 @@ func TestGenerator(t *testing.T) {
 			),
 		},
 	} {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+
 			generator := xmlstruct.NewGenerator(tc.options...)
 			if tc.xmlStr != "" {
 				assert.NoError(t, generator.ObserveReader(strings.NewReader(tc.xmlStr)))

internal/tests/aixm/aixm_test.go

@@ -13,6 +13,8 @@ import (
 )
 
 func TestAIXM(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"note":            "NoteLower",            // Disambiguate between Note and note.

internal/tests/gml/gml_test.go

@@ -14,6 +14,8 @@ import (
 )
 
 func TestGML(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"note": "LowerNote",

internal/tests/gpx/gpx_test.go

@@ -14,6 +14,8 @@ import (
 )
 
 func TestGPX(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"gpx":     "GPX",

internal/tests/interlis/interlis_test.go

@@ -11,6 +11,8 @@ import (
 )
 
 func TestInterlis(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"BOUNDARY":      "Boundary",

internal/tests/osm/osm_test.go

@@ -11,6 +11,8 @@ import (
 )
 
 func TestOSM(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"osm":    "OSM",

internal/tests/play/play_test.go

@@ -11,6 +11,8 @@ import (
 )
 
 func TestPlay(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"ACT":      "Act",

internal/tests/xsd/xsd_test.go

@@ -13,6 +13,8 @@ import (
 )
 
 func TestXSD(t *testing.T) {
+	t.Parallel()
+
 	generator := xmlstruct.NewGenerator(
 		xmlstruct.WithExportRenames(map[string]string{
 			"appinfo": "AppInfo",

xmlstruct_test.go

@@ -8,6 +8,8 @@ import (
 )
 
 func TestDefaultExportNameFunc(t *testing.T) {
+	t.Parallel()
+
 	for _, tc := range []struct {
 		localName string
 		expected  string
@@ -49,7 +51,10 @@ func TestDefaultExportNameFunc(t *testing.T) {
 			expected:  "SnakeID",
 		},
 	} {
+		tc := tc
 		t.Run(tc.localName, func(t *testing.T) {
+			t.Parallel()
+
 			xmlName := xml.Name{
 				Local: tc.localName,
 			}