fix: update Go version to 1.25.0 and remove toolchain directive (#151)

Author: KoheiMatsuno99

.github/workflows/lint.yaml

@@ -21,7 +21,7 @@ jobs:
         with:
           go-version: 'stable'
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v5
+        uses: golangci/golangci-lint-action@v7
         with:
           version: latest
           github-token: ${{ secrets.GITHUB_TOKEN }}

.golangci.yaml

@@ -1,116 +1,94 @@
----
+version: "2"
 linters:
   enable:
-    # check when errors are compared without errors.Is
+    - dupword
     - errorlint
-
-    # check imports order and makes it always deterministic.
-    - gci
-
-    # Very Basic spell error checker
+    - mirror
     - misspell
-
-    # Fast, configurable, extensible, flexible, and beautiful linter for Go.
-    # Drop-in replacement of golint.
-    - revive
-
-    # make sure to use t.Helper() when needed
-    - thelper
-
-    # ensure that lint exceptions have explanations. Consider the case below:
     - nolintlint
-
-    # detect duplicated words in code
-    - dupword
-
-    # mirror suggests rewrites to avoid unnecessary []byte/string conversion
-    - mirror
-
-    # testify checks good usage of github.com/stretchr/testify.
+    - revive
     - testifylint
-
-linters-settings:
-  dupword:
-    # Keywords used to ignore detection.
-    # Default: []
-    ignore:
-      - "FAIL" # "FAIL FAIL" is tolerated
-
-  nolintlint:
-    # Disable to ensure that all nolint directives actually have an effect.
-    # Default: false
-    allow-unused: true  # too many false positive reported
-    # Exclude following linters from requiring an explanation.
-    # Default: []
-    allow-no-explanation: []
-    # Enable to require an explanation of nonzero length
-    # after each nolint directive.
-    # Default: false
-    require-explanation: true
-    # Enable to require nolint directives to mention the specific
-    # linter being suppressed.
-    # Default: false
-    require-specific: true
-
-  revive:
-    rules:
-      - name: bare-return
-      - name: blank-imports
-      - name: comment-spacings
-      - name: context-as-argument
-        arguments:
-          - allowTypesBefore: "*testing.T"
-      - name: context-keys-type
-      - name: defer
-        arguments:
-          - ["call-chain", "loop"]
-      - name: dot-imports
-      - name: early-return
-      - name: empty-block
-      - name: error-return
-      - name: error-strings
-      - name: error-naming
-      - name: errorf
-      - name: exported
-        arguments:
-          # enables checking public methods of private types
-          - "checkPrivateReceivers"
-          # make error messages clearer
-          - "sayRepetitiveInsteadOfStutters"
-      - name: if-return
-      - name: import-shadowing
-      - name: increment-decrement
-      - name: indent-error-flow
-      - name: exported
-      - name: var-naming
-      - name: var-declaration
-      - name: package-comments
-      - name: range
-      - name: receiver-naming
-      - name: redefines-builtin-id
-      - name: superfluous-else
-      - name: time-naming
-      - name: time-equal
-      - name: unexported-return
-      - name: use-any
-      - name: unreachable-code
-      - name: unhandled-error
-        arguments:
-          - "fmt.Print.*"
-          - "fmt.Fprint.*"
-          - "bytes.Buffer.Write.*"
-          - "strings.Builder.Write.*"
-      - name: unused-parameter
-      - name: unused-receiver
-      - name: useless-break
-
-  # define the import orders
-  gci:
-    sections:
-      # Standard section: captures all standard packages.
-      - standard
-      # Default section: catchall that is not standard or custom
-      - default
-      # Custom section: groups all imports with the specified Prefix.
-      - prefix(github.com/mfridman)
-
+    - thelper
+  settings:
+    dupword:
+      ignore:
+        - FAIL
+    nolintlint:
+      require-explanation: true
+      require-specific: true
+      allow-unused: true
+    revive:
+      rules:
+        - name: bare-return
+        - name: blank-imports
+        - name: comment-spacings
+        - name: context-as-argument
+          arguments:
+            - allowTypesBefore: '*testing.T'
+        - name: context-keys-type
+        - name: defer
+          arguments:
+            - - call-chain
+              - loop
+        - name: dot-imports
+        - name: early-return
+        - name: empty-block
+        - name: error-return
+        - name: error-strings
+        - name: error-naming
+        - name: errorf
+        - name: exported
+          arguments:
+            - checkPrivateReceivers
+            - sayRepetitiveInsteadOfStutters
+        - name: if-return
+        - name: import-shadowing
+        - name: increment-decrement
+        - name: indent-error-flow
+        - name: exported
+        - name: var-declaration
+        - name: package-comments
+        - name: range
+        - name: receiver-naming
+        - name: redefines-builtin-id
+        - name: superfluous-else
+        - name: time-naming
+        - name: time-equal
+        - name: unexported-return
+        - name: use-any
+        - name: unreachable-code
+        - name: unhandled-error
+          arguments:
+            - fmt.Print.*
+            - fmt.Fprint.*
+            - bytes.Buffer.Write.*
+            - strings.Builder.Write.*
+        - name: unused-parameter
+        - name: unused-receiver
+        - name: useless-break
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gci
+  settings:
+    gci:
+      sections:
+        - standard
+        - default
+        - prefix(github.com/mfridman)
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

go.mod

@@ -1,8 +1,6 @@
 module github.com/mfridman/tparse
 
-go 1.23.0
-
-toolchain go1.24.2
+go 1.25.0
 
 require (
 	github.com/charmbracelet/lipgloss v1.1.0

tests/cached_test.go

@@ -62,7 +62,7 @@ func TestPackageCache(t *testing.T) {
 
 			summary, err := parse.Process(f)
 			require.NoError(t, err)
-			assert.Equal(t, len(summary.Packages), len(tc.expected))
+			assert.Len(t, summary.Packages, len(tc.expected))
 
 			for name, pkg := range summary.Packages {
 				t.Run(name, func(t *testing.T) {

tests/cover_test.go

@@ -65,7 +65,7 @@ func Test(t *testing.T) {
 
 			summary, err := parse.Process(f)
 			require.NoError(t, err)
-			assert.Equal(t, len(summary.Packages), len(tc.expected))
+			assert.Len(t, summary.Packages, len(tc.expected))
 
 			for name, pkg := range summary.Packages {
 				t.Run(name, func(t *testing.T) {

tests/outcome_test.go

@@ -62,7 +62,7 @@ func TestFinalOutcome(t *testing.T) {
 			}
 			summary, err := parse.Process(f)
 			require.NoError(t, err)
-			assert.Equal(t, len(summary.Packages), len(tc.registry))
+			assert.Len(t, summary.Packages, len(tc.registry))
 			assert.Equal(t, summary.ExitCode(), tc.exitCode)
 
 			for name, pkg := range summary.Packages {

tests/package_start_test.go

@@ -37,7 +37,7 @@ func TestPackageStartTime(t *testing.T) {
 
 	summary, err := parse.Process(f)
 	require.NoError(t, err)
-	assert.Equal(t, len(summary.Packages), len(expected))
+	assert.Len(t, summary.Packages, len(expected))
 
 	for _, p := range summary.Packages {
 		if p.StartTime.IsZero() {

tests/race_test.go

@@ -73,7 +73,7 @@ func TestRaceDetected(t *testing.T) {
 				if !ok {
 					t.Fatalf("failed to find package: %q", name)
 				}
-				assert.Equal(t, len(pkg.DataRaceTests), len(wantTestName))
+				assert.Len(t, pkg.DataRaceTests, len(wantTestName))
 				if len(pkg.DataRaceTests) > 0 {
 					for i := range pkg.DataRaceTests {
 						assert.Equal(t, pkg.DataRaceTests[i], wantTestName[i])

tests/sort_test.go

@@ -58,7 +58,7 @@ func TestSortName(t *testing.T) {
 
 			summary, err := parse.Process(f)
 			require.NoError(t, err)
-			assert.Equal(t, len(summary.Packages), len(tc.expected))
+			assert.Len(t, summary.Packages, len(tc.expected))
 			packages := summary.GetSortedPackages(parse.SortByPackageName)
 
 			for i, pkg := range packages {
@@ -120,7 +120,7 @@ func TestSortCoverage(t *testing.T) {
 
 			summary, err := parse.Process(f)
 			require.NoError(t, err)
-			assert.Equal(t, len(summary.Packages), len(tc.expected))
+			assert.Len(t, summary.Packages, len(tc.expected))
 			packages := summary.GetSortedPackages(parse.SortByCoverage)
 
 			for i, pkg := range packages {
@@ -182,7 +182,7 @@ func TestSortElapsed(t *testing.T) {
 
 			summary, err := parse.Process(f)
 			require.NoError(t, err)
-			assert.Equal(t, len(summary.Packages), len(tc.expected))
+			assert.Len(t, summary.Packages, len(tc.expected))
 			packages := summary.GetSortedPackages(parse.SortByElapsed)
 
 			for i, pkg := range packages {