Update Go (#137)

* Update minimum Go version to 1.24.0.
* Update Go matrix to current releses.
* Run Go `modernize` tool.
* Update golangci-lint.
* Fix linting issues.

Signed-off-by: SuperQ <superq@gmail.com>

Author: SuperQ

.github/workflows/static-analysis.yml

@@ -18,8 +18,8 @@ jobs:
     strategy:
       matrix:
         go-version:
-        - "1.23"
         - "1.24"
+        - "1.25"
     runs-on: ubuntu-latest
 
     steps:
@@ -36,4 +36,4 @@ jobs:
         uses: golangci/golangci-lint-action@4afd733a84b1f43292c63897423277bb7f4313a9 # v8.0.0
         with:
           args: --verbose
-          version: v2.2.1
+          version: v2.6.0

.github/workflows/test.yml

@@ -14,8 +14,8 @@ jobs:
       fail-fast: false
       matrix:
         go-version:
-        - "1.23"
         - "1.24"
+        - "1.25"
         os: [ubuntu-latest, macos-latest]
     runs-on: ${{ matrix.os }}
 

.golangci.yml

@@ -9,14 +9,6 @@ linters:
       - common-false-positives
       - legacy
       - std-error-handling
-    paths:
-      - third_party$
-      - builtin$
-      - examples$
 formatters:
   exclusions:
     generated: lax
-    paths:
-      - third_party$
-      - builtin$
-      - examples$

client_linux.go

@@ -1,5 +1,4 @@
 //go:build linux
-// +build linux
 
 package wifi
 
@@ -964,7 +963,7 @@ func decodeRSN(b []byte) (*RSNInfo, error) {
 	}
 
 	ri.PairwiseCiphers = make([]RSNCipher, 0, pcCount) // Pre-allocate with known capacity
-	for i := 0; i < pcCount; i++ {
+	for range pcCount {
 		sel := binary.BigEndian.Uint32(b[pos : pos+4])
 		ri.PairwiseCiphers = append(ri.PairwiseCiphers, RSNCipher(sel))
 		pos += 4
@@ -992,7 +991,7 @@ func decodeRSN(b []byte) (*RSNInfo, error) {
 	}
 
 	ri.AKMs = make([]RSNAKM, 0, akmCount) // Pre-allocate with known capacity
-	for i := 0; i < akmCount; i++ {
+	for range akmCount {
 		sel := binary.BigEndian.Uint32(b[pos : pos+4])
 		ri.AKMs = append(ri.AKMs, RSNAKM(sel))
 		pos += 4

client_linux_integration_test.go

@@ -1,5 +1,4 @@
 //go:build linux
-// +build linux
 
 package wifi_test
 
@@ -46,7 +45,7 @@ func TestIntegrationLinuxConcurrent(t *testing.T) {
 	wg.Add(workers)
 	defer wg.Wait()
 
-	for i := 0; i < workers; i++ {
+	for i := range workers {
 		go func(differentI int) {
 			defer wg.Done()
 			execN(t, iterations, names, differentI)
@@ -58,7 +57,7 @@ func execN(t *testing.T, n int, expect []string, workerID int) {
 	c := testClient(t)
 
 	names := make(map[string]int)
-	for i := 0; i < n; i++ {
+	for i := range n {
 		ifis, err := c.Interfaces()
 		if err != nil {
 			panicf("[worker_id %d; iteration %d] failed to retrieve interfaces: %v", workerID, i, err)
@@ -112,7 +111,7 @@ func testClient(t *testing.T) *wifi.Client {
 	return c
 }
 
-func panicf(format string, a ...interface{}) {
+func panicf(format string, a ...any) {
 	panic(fmt.Sprintf(format, a...))
 }
 

client_linux_test.go

@@ -1,5 +1,4 @@
 //go:build linux
-// +build linux
 
 package wifi
 
@@ -568,7 +567,7 @@ func bitrateAttr(bitrate int) uint32 {
 	return uint32(bitrate / 100 / 1000)
 }
 
-func mustMessages(t *testing.T, command uint8, want interface{}) genltest.Func {
+func mustMessages(t *testing.T, command uint8, want any) genltest.Func {
 	var as []attributeser
 
 	switch xs := want.(type) {

client_others.go

@@ -1,5 +1,4 @@
 //go:build !linux
-// +build !linux
 
 package wifi
 

go.mod

@@ -1,6 +1,6 @@
 module github.com/mdlayher/wifi
 
-go 1.23.0
+go 1.24.0
 
 require (
 	github.com/google/go-cmp v0.7.0