Update linter

Author: zivkovicmilos

.github/golangci.yaml

@@ -7,8 +7,6 @@ run:
   modules-download-mode: readonly
   allow-parallel-runners: false
   go: ""
-  skip-dirs:
-    - serve/filters/mocks
 
 output:
   uniq-by-line: false
@@ -25,6 +23,8 @@ issues:
       linters:
         - nilnil
         - gosec
+  exclude-dirs:
+    - serve/filters/mocks
 
 linters:
   fast: false
@@ -35,7 +35,7 @@ linters:
     - bidichk # Checks for dangerous unicode character sequences
     - durationcheck # Check for two durations multiplied together
     - errcheck # Forces to not skip error check
-    - exportloopref # Checks for pointers to enclosing loop variables
+    - copyloopvar # Checks for pointers to enclosing loop variables
     - gocritic # Bundles different linting checks
     - godot # Checks for periods at the end of comments
     - gomoddirectives # Allow or ban replace directives in go.mod
@@ -124,3 +124,6 @@ linters-settings:
       rules:
         json: goCamel
         yaml: goCamel
+  gosec:
+    excludes:
+      - G115 # G115: integer overflow conversion uint64 -> int6

cmd/waiter.go

@@ -53,8 +53,6 @@ func (w *waiter) wait() error {
 	})
 
 	for _, fn := range w.waitFns {
-		fn := fn
-
 		g.Go(
 			func() error {
 				return fn(ctx)

events/subscription_test.go

@@ -39,8 +39,6 @@ func TestSubscription_EventSupported(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 

fetch/slots_test.go

@@ -146,8 +146,6 @@ func TestSlots_FindGaps(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 

go.mod

@@ -2,8 +2,6 @@ module github.com/gnolang/tx-indexer
 
 go 1.22
 
-toolchain go1.22.5
-
 require (
 	github.com/99designs/gqlgen v0.17.49
 	github.com/cockroachdb/pebble v1.1.2

serve/filters/filter/base_test.go

@@ -20,8 +20,6 @@ func TestBaseFilter_GetType(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 

serve/filters/filter/block_test.go

@@ -38,8 +38,6 @@ func TestBlockFilter_GetChanges(t *testing.T) {
 
 	// Update the block filter with dummy blocks
 	for _, block := range blocks {
-		block := block
-
 		f.UpdateWith(block)
 	}
 

serve/filters/filter/tx_test.go

@@ -221,7 +221,6 @@ func TestApplyFilters(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		tt := tt
 		t.Run(tt.name, func(t *testing.T) {
 			t.Parallel()
 
@@ -295,7 +294,6 @@ func TestApplyFiltersWithLargeData(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		tt := tt
 		t.Run(tt.name, func(t *testing.T) {
 			t.Parallel()
 

serve/filters/manager_test.go

@@ -99,8 +99,6 @@ func Test_NewBlockEvents(t *testing.T) {
 	defer filterManager.UninstallFilter(id)
 
 	for _, block := range blocks {
-		block := block
-
 		blockCh <- &types.NewBlock{
 			Block: block,
 		}

serve/filters/subscription.go

@@ -61,7 +61,6 @@ func (sm *subscriptionMap) sendEvent(eventType events.Type, data any) {
 	}
 
 	for id, sub := range sm.subscriptions {
-		sub := sub
 		if sub.GetType() != eventType {
 			continue
 		}

serve/handler_test.go

@@ -99,8 +99,6 @@ func TestHTTP_Handle(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 

serve/handlers/block/block_test.go

@@ -33,8 +33,6 @@ func TestGetBlock_InvalidParams(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 

serve/handlers/subs/subs_test.go

@@ -108,8 +108,6 @@ func TestUninstallFilter_InvalidParams(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 
@@ -211,8 +209,6 @@ func TestGetFilterChanges_InvalidParams(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 
@@ -546,8 +542,6 @@ func TestSubscribeUnsubscribe_InvalidParams(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()
 

serve/handlers/tx/tx_test.go

@@ -32,8 +32,6 @@ func TestGetTx_InvalidParams(t *testing.T) {
 	}
 
 	for _, testCase := range testTable {
-		testCase := testCase
-
 		t.Run(testCase.name, func(t *testing.T) {
 			t.Parallel()