Update 47

.github/workflows/buildtest.yaml

@@ -35,7 +35,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: "1.23.3"
+          go-version: "1.23.7"
         env:
           GOOS: ${{ matrix.targetos }}
           GOARCH: ${{ matrix.arch }}
@@ -66,7 +66,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: "1.23.3"
+          go-version: "1.23.7"
         env:
           GOOS: ${{ matrix.targetos }}
           GOARCH: ${{ matrix.arch }}
@@ -117,6 +117,6 @@ jobs:
             ${{ runner.os }}-go-
       - uses: actions/setup-go@v5
         with:
-          go-version: "1.23.3"
+          go-version: "1.23.7"
       - name: Run simulation tests
         run: make test-sim-ci

.github/workflows/codeql.yml

@@ -24,7 +24,7 @@ jobs:
         uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: 1.23.3
+          go-version: 1.23.7
       # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
         uses: github/codeql-action/init@v3

.github/workflows/devskim.yml

@@ -7,7 +7,7 @@ name: DevSkim
 
 on:
   push:
-    branches: [ "main", "develop", "feat*", "main*", "unstable*" ]
+    branches: [ "main" ]
   pull_request:
     branches: [ "main" ]
   schedule:
@@ -16,7 +16,7 @@ on:
 jobs:
   lint:
     name: DevSkim
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-22.04
     permissions:
       actions: read
       contents: read
@@ -27,6 +27,8 @@ jobs:
 
       - name: Run DevSkim scanner
         uses: microsoft/DevSkim-Action@v1
+        with:
+          ignore-globs: "third-party-chains/*"
 
       - name: Upload DevSkim scan results to GitHub Security tab
         uses: github/codeql-action/upload-sarif@v3

.github/workflows/docker.yaml

@@ -26,7 +26,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: "1.23.3"
+          go-version: "1.23.7"
 
       - name: Setup Golang caches
         uses: actions/cache@v4

.github/workflows/golangci.yml

@@ -23,12 +23,12 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: "1.23.3"
+          go-version: "1.23.7"
           cache: false
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
-          version: v1.64.5
+          version: v2.0.2
           args: --timeout 15m
           # Optional: working directory, useful for monorepos
           # working-directory: somedir

.github/workflows/icq-relayer-build-test.yml

@@ -11,7 +11,7 @@ jobs:
   test:
     strategy:
       matrix:
-        go-version: [1.23.3]
+        go-version: [1.23.7]
         os: [ubuntu-latest, macos-latest]
     runs-on: ${{ matrix.os }}
     steps:
@@ -25,7 +25,7 @@ jobs:
   build:
     strategy:
       matrix:
-        go-version: [1.23.3]
+        go-version: [1.23.7]
         os: [ubuntu-latest, macos-latest]
     runs-on: ${{ matrix.os }}
     steps:

.github/workflows/simulation.yml

@@ -19,7 +19,7 @@ jobs:
             ${{ runner.os }}-go-
       - uses: actions/setup-go@v5
         with:
-          go-version: "1.23.3"
+          go-version: "1.23.7"
 
       - name: Run simulation tests
         run: make test-sim-non-determinism

.golangci.yml

@@ -1,125 +1,133 @@
+version: "2"
 run:
   tests: true
-  #   # timeout for analysis, e.g. 30s, 5m, default is 1m
-  timeout: 10m
-
 linters:
-  disable-all: true
+  default: none
   enable:
     - copyloopvar
     - errcheck
-    - gci
     - goconst
     - gocritic
-    - gofumpt
     - gosec
-    - gosimple
     - govet
     - ineffassign
     - misspell
     - nakedret
+    - revive
     - staticcheck
     - thelper
-    - typecheck
-    - stylecheck
-    - revive
-    - typecheck
-    - usetesting
     - unconvert
-    - unparam # Prefer unparam over revive's unused param. It is more thorough in its checking.
+    - unparam
     - unused
-    - misspell
-
+    - usetesting
+  settings:
+    revive:
+      enable-all-rules: true
+      rules:
+        - name: use-any
+          disabled: true
+        - name: if-return
+          disabled: true
+        - name: max-public-structs
+          disabled: true
+        - name: unchecked-type-assertion
+          disabled: true
+        - name: import-shadowing
+          disabled: true
+        - name: cognitive-complexity
+          disabled: true
+        - name: var-naming
+          disabled: true
+        - name: argument-limit
+          disabled: true
+        - name: cyclomatic
+          disabled: true
+        - name: file-header
+          disabled: true
+        - name: function-length
+          disabled: true
+        - name: function-result-limit
+          disabled: true
+        - name: line-length-limit
+          disabled: true
+        - name: flag-parameter
+          disabled: true
+        - name: add-constant
+          disabled: true
+        - name: empty-lines
+          disabled: true
+        - name: unused-receiver
+          disabled: true
+        - name: banned-characters
+          disabled: true
+        - name: deep-exit
+          disabled: true
+        - name: confusing-results
+          disabled: true
+        - name: unused-parameter
+          disabled: true
+        - name: modifies-value-receiver
+          disabled: true
+        - name: early-return
+          disabled: true
+        - name: confusing-naming
+          disabled: true
+        - name: defer
+          disabled: true
+        - name: unused-parameter
+          disabled: true
+        - name: unhandled-error
+          arguments:
+            - b.WriteString
+            - respJSON.Write
+            - fmt.Printf
+            - fmt.Print
+            - fmt.Println
+            - fmt.Fprintf
+          disabled: true
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - revive
+        text: differs only by capitalization to method
+      - linters:
+          - gosec
+        text: Use of weak random number generator
+    paths:
+      - third-party-chains
+      - third_party$
+      - builtin$
+      - examples$
 issues:
-  exclude-rules:
-    - text: "differs only by capitalization to method"
-      linters:
-        - revive
-    - text: "Use of weak random number generator"
-      linters:
-        - gosec
-
   max-issues-per-linter: 10000
   max-same-issues: 10000
-  exclude-dirs:
-    - third-party-chains
-
-linters-settings:
-  gci:
-    sections:
-      - standard # Standard section: captures all standard packages.
-      - default # Default section: contains all imports that could not be matched to another section type.
-      - blank # blank imports
-      - dot # dot imports
-      - prefix(cosmossdk.io)
-      - prefix(github.com/cosmos/cosmos-sdk)
-      - prefix(github.com/cometbft/cometbft)
-      - prefix(github.com/cosmos/ibc-go)
-      - prefix(github.com/quicksilver-zone/quicksilver)
-    custom-order: true
-  revive:
-    enable-all-rules: true
-    # Do NOT whine about the following, full explanation found in:
-    # https://github.com/mgechev/revive/blob/master/RULES_DESCRIPTIONS.md#description-of-available-rules
-    rules:
-      - name: use-any
-        disabled: true
-      - name: if-return
-        disabled: true
-      - name: max-public-structs
-        disabled: true
-      - name: unchecked-type-assertion
-        disabled: true
-      - name: import-shadowing
-        disabled: true
-      - name: cognitive-complexity
-        disabled: true
-      - name: var-naming
-        disabled: true
-      - name: argument-limit
-        disabled: true
-      - name: cyclomatic
-        disabled: true
-      - name: file-header
-        disabled: true
-      - name: function-length
-        disabled: true
-      - name: function-result-limit
-        disabled: true
-      - name: line-length-limit
-        disabled: true
-      - name: flag-parameter
-        disabled: true
-      - name: add-constant
-        disabled: true
-      - name: empty-lines
-        disabled: true
-      - name: unused-receiver # remove later
-        disabled: true
-      - name: banned-characters
-        disabled: true
-      - name: deep-exit
-        disabled: true
-      - name: confusing-results
-        disabled: true
-      - name: unused-parameter
-        disabled: true
-      - name: modifies-value-receiver
-        disabled: true
-      - name: early-return
-        disabled: true
-      - name: confusing-naming
-        disabled: true
-      - name: defer
-        disabled: true
-      - name: unused-parameter # Disabled in favour of unparam.
-        disabled: true
-      - name: unhandled-error # enable later, currently covered by errcheck
-        disabled: true
-        arguments:
-          - "b.WriteString"
-          - "respJSON.Write"
-          - "fmt.Printf"
-          - "fmt.Print"
-          - "fmt.Println"
-          - "fmt.Fprintf"
+formatters:
+  enable:
+    - gci
+    - gofumpt
+  settings:
+    gci:
+      sections:
+        - standard
+        - default
+        - blank
+        - dot
+        - prefix(cosmossdk.io)
+        - prefix(github.com/cosmos/cosmos-sdk)
+        - prefix(github.com/cometbft/cometbft)
+        - prefix(github.com/cosmos/ibc-go)
+        - prefix(github.com/quicksilver-zone/quicksilver)
+      custom-order: true
+  exclusions:
+    generated: lax
+    paths:
+      - third-party-chains
+      - third_party$
+      - builtin$
+      - examples$