chore(ci): fix code for ci linter

Signed-off-by: Minh Huy Tran <huy@perun.network>

.github/workflows/ci.yml

@@ -36,10 +36,13 @@ jobs:
         with:
           go-version: ${{ env.go-version }}
 
+      - name: Copy .golangci.yml
+        run: cp ../../.golangci.yml .
+
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v3
         with:
-          version: v1.55
+          version: v1.56.2
 
       - name: Lint proto files
         uses: plexsystems/protolint-action@v0.7.0

.golangci.yml

@@ -39,6 +39,9 @@ linters:
     - scopelint
     - golint
     - interfacer
+    - varcheck
+    - structcheck
+    - deadcode
 
 linters-settings:
   cyclop:

wire/net/endpoint_registry_internal_test.go

@@ -111,8 +111,6 @@ func nilConsumer(wire.Address) wire.Consumer { return nil }
 // dialed in the background. It also tests that the dialing process combines
 // with the Listener, so that if a connection to a peer that is still being
 // dialed comes in, the peer is assigned that connection.
-//
-//nolint:testify
 func TestRegistry_Get(t *testing.T) {
 	t.Parallel()
 	rng := test.Prng(t)

wire/net/simple/dialer_internal_test.go

@@ -103,11 +103,11 @@ func TestDialer_Dial(t *testing.T) {
 		ct := test.NewConcurrent(t)
 		go ct.Stage("accept", func(rt test.ConcT) {
 			conn, err := l.Accept(ser)
-			require.NoError(t, err)  //nolint: testifylint
-			require.NotNil(rt, conn) //nolint: testifylint
+			require.NoError(t, err)  //nolint:testifylint
+			require.NotNil(rt, conn) //nolint:testifylint
 
 			re, err := conn.Recv()
-			require.NoError(t, err) //nolint: testifylint
+			require.NoError(t, err) //nolint:testifylint
 			assert.Equal(t, re, e)
 		})
 

wire/net/test/connhub_internal_test.go

@@ -46,9 +46,9 @@ func TestConnHub_Create(t *testing.T) {
 		go ctxtest.AssertTerminates(t, timeout, func() {
 			ct.Stage("accept", func(rt pkgtest.ConcT) {
 				conn, err := l.Accept(ser)
-				require.NoError(t, err)                                                           //nolint: testifylint
-				require.NotNil(rt, conn)                                                          //nolint: testifylint
-				require.NoError(t, conn.Send(wiretest.NewRandomEnvelope(rng, wire.NewPingMsg()))) //nolint: testifylint
+				require.NoError(t, err)                                                           //nolint:testifylint
+				require.NotNil(rt, conn)                                                          //nolint:testifylint
+				require.NoError(t, conn.Send(wiretest.NewRandomEnvelope(rng, wire.NewPingMsg()))) //nolint:testifylint
 			})
 		})