sync: update CI config files

.github/workflows/go-check.yml

@@ -11,12 +11,12 @@ jobs:
     env:
       RUNGOGENERATE: false
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
         with:
           submodules: recursive
-      - uses: actions/setup-go@v2
+      - uses: actions/setup-go@v3
         with:
-          go-version: "1.18.x"
+          go-version: "1.19.x"
       - name: Run repo-specific setup
         uses: ./.github/actions/go-check-setup
         if: hashFiles('./.github/actions/go-check-setup') != ''
@@ -27,7 +27,7 @@ jobs:
             echo "RUNGOGENERATE=true" >> $GITHUB_ENV
           fi
       - name: Install staticcheck
-        run: go install honnef.co/go/tools/cmd/staticcheck@d7e217c1ff411395475b2971c0824e1e7cc1af98 # 2022.1 (v0.3.0)
+        run: go install honnef.co/go/tools/cmd/staticcheck@376210a89477dedbe6fdc4484b233998650d7b3c # 2022.1.3 (v0.3.3)
       - name: Check that go.mod is tidy
         uses: protocol/multiple-go-modules@v1.2
         with:

.github/workflows/go-test.yml

@@ -10,16 +10,16 @@ jobs:
       fail-fast: false
       matrix:
         os: [ "ubuntu", "windows", "macos" ]
-        go: [ "1.17.x", "1.18.x" ]
+        go: [ "1.18.x", "1.19.x" ]
     env:
       COVERAGES: ""
     runs-on: ${{ format('{0}-latest', matrix.os) }}
     name: ${{ matrix.os }} (go ${{ matrix.go }})
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
         with:
           submodules: recursive
-      - uses: actions/setup-go@v2
+      - uses: actions/setup-go@v3
         with:
           go-version: ${{ matrix.go }}
       - name: Go information
@@ -43,7 +43,7 @@ jobs:
           # Use -coverpkg=./..., so that we include cross-package coverage.
           # If package ./A imports ./B, and ./A's tests also cover ./B,
           # this means ./B's coverage will be significantly higher than 0%.
-          run: go test -v -coverprofile=module-coverage.txt -coverpkg=./... ./...
+          run: go test -v -shuffle=on -coverprofile=module-coverage.txt -coverpkg=./... ./...
       - name: Run tests (32 bit)
         if: ${{ matrix.os != 'macos' }} # can't run 32 bit tests on OSX.
         uses: protocol/multiple-go-modules@v1.2
@@ -52,7 +52,7 @@ jobs:
         with:
           run: |
             export "PATH=${{ env.PATH_386 }}:$PATH"
-            go test -v ./...
+            go test -v -shuffle=on ./...
       - name: Run tests with race detector
         if: ${{ matrix.os == 'ubuntu' }} # speed things up. Windows and OSX VMs are slow
         uses: protocol/multiple-go-modules@v1.2
@@ -62,7 +62,7 @@ jobs:
         shell: bash
         run: echo "COVERAGES=$(find . -type f -name 'module-coverage.txt' | tr -s '\n' ',' | sed 's/,$//')" >> $GITHUB_ENV
       - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@f32b3a3741e1053eb607407145bc9619351dc93b # v2.1.0
+        uses: codecov/codecov-action@81cd2dc8148241f03f5839d295e000b8f761e378 # v3.1.0
         with:
           files: '${{ env.COVERAGES }}'
           env_vars: OS=${{ matrix.os }}, GO=${{ matrix.go }}

dht.go

@@ -625,16 +625,23 @@ func (dht *IpfsDHT) rtPeerLoop(proc goprocess.Process) {
 // peerFound signals the routingTable that we've found a peer that
 // might support the DHT protocol.
 // If we have a connection a peer but no exchange of a query RPC ->
-//    LastQueriedAt=time.Now (so we don't ping it for some time for a liveliness check)
-//    LastUsefulAt=0
+//
+//	LastQueriedAt=time.Now (so we don't ping it for some time for a liveliness check)
+//	LastUsefulAt=0
+//
 // If we connect to a peer and then exchange a query RPC ->
-//    LastQueriedAt=time.Now (same reason as above)
-//    LastUsefulAt=time.Now (so we give it some life in the RT without immediately evicting it)
+//
+//	LastQueriedAt=time.Now (same reason as above)
+//	LastUsefulAt=time.Now (so we give it some life in the RT without immediately evicting it)
+//
 // If we query a peer we already have in our Routing Table ->
-//    LastQueriedAt=time.Now()
-//    LastUsefulAt remains unchanged
+//
+//	LastQueriedAt=time.Now()
+//	LastUsefulAt remains unchanged
+//
 // If we connect to a peer we already have in the RT but do not exchange a query (rare)
-//    Do Nothing.
+//
+//	Do Nothing.
 func (dht *IpfsDHT) peerFound(ctx context.Context, p peer.ID, queryPeer bool) {
 	if c := baseLogger.Check(zap.DebugLevel, "peer found"); c != nil {
 		c.Write(zap.String("peer", p.String()))

dht_options.go

@@ -65,9 +65,9 @@ func RoutingTableRefreshQueryTimeout(timeout time.Duration) Option {
 // RoutingTableRefreshPeriod sets the period for refreshing buckets in the
 // routing table. The DHT will refresh buckets every period by:
 //
-// 1. First searching for nearby peers to figure out how many buckets we should try to fill.
-// 1. Then searching for a random key in each bucket that hasn't been queried in
-//    the last refresh period.
+//  1. First searching for nearby peers to figure out how many buckets we should try to fill.
+//  1. Then searching for a random key in each bucket that hasn't been queried in
+//     the last refresh period.
 func RoutingTableRefreshPeriod(period time.Duration) Option {
 	return func(c *dhtcfg.Config) error {
 		c.RoutingTable.RefreshInterval = period

fullrt/options.go

@@ -2,6 +2,7 @@ package fullrt
 
 import (
 	"fmt"
+
 	kaddht "github.com/libp2p/go-libp2p-kad-dht"
 )
 

go.mod

@@ -1,6 +1,6 @@
 module github.com/libp2p/go-libp2p-kad-dht
 
-go 1.17
+go 1.18
 
 require (
 	github.com/gogo/protobuf v1.3.2