chore: upgrade component versions (#33)

Author: joshdk

.github/workflows/main.yml

@@ -18,7 +18,7 @@ jobs:
       - name: Setup go
         uses: actions/setup-go@v2
         with:
-          go-version: 1.17
+          go-version: 1.19
 
       - name: Checkout code
         uses: actions/checkout@v2
@@ -37,7 +37,7 @@ jobs:
       - name: Setup go
         uses: actions/setup-go@v2
         with:
-          go-version: 1.17
+          go-version: 1.19
 
       - name: Checkout code
         uses: actions/checkout@v2
@@ -53,12 +53,13 @@ jobs:
       - name: Setup go
         uses: actions/setup-go@v2
         with:
-          go-version: 1.17
+          go-version: 1.19
 
       - name: Checkout code
         uses: actions/checkout@v2
 
       - name: Run golangci-lint
         uses: golangci/golangci-lint-action@v2
         with:
-          version: v1.44.0
+          # https://github.com/golangci/golangci-lint/releases/tag/v1.50.1
+          version: v1.50.1

.github/workflows/release.yml

@@ -15,7 +15,7 @@ jobs:
       - name: Setup go
         uses: actions/setup-go@v2
         with:
-          go-version: 1.17
+          go-version: 1.19
 
       - name: Checkout code
         uses: actions/checkout@v2

.golangci.yml

@@ -1,26 +1,38 @@
+issues:
+  exclude-use-default: true
+  exclude:
+    # Triggered by long table tests.
+    - Function 'Test\w+' is too long
+
 linters:
   enable-all: true
   disable:
-    - cyclop
+    # Linters that are deprecated.
     - exhaustivestruct
-    - forbidigo
+    - scopelint
+    - interfacer
+    - maligned
+    - golint
+    - ifshort
+    - structcheck
+    - nosnakecase
+    - deadcode
+    - varcheck
+
+    # Linters that are disabled because of generics.
+    - rowserrcheck
+    - sqlclosecheck
+    - wastedassign
+
+    # Linters that are not used for this project.
+    - cyclop
+    - exhaustruct
     - goerr113
     - gomnd
     - gosec
-    - lll
-    - maligned
     - nlreturn
     - paralleltest
     - testpackage
     - tparallel
     - wrapcheck
     - wsl
-
-issues:
-  exclude-use-default: true
-  exclude:
-    # Triggered by table tests calling t.Run. See
-    # https://github.com/kyoh86/scopelint/issues/4 for more information.
-    - Using the variable on range scope `test` in function literal
-    # Triggered by long table tests.
-    - Function 'Test\w+' is too long

go.mod

@@ -1,11 +1,11 @@
 module github.com/joshdk/retry
 
-go 1.17
+go 1.19
 
-require github.com/stretchr/testify v1.7.0
+require github.com/stretchr/testify v1.8.1
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/pmezard/go-difflib v1.0.0 // indirect
-	gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b // indirect
+	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

go.sum

@@ -4,10 +4,14 @@ github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSs
 github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
 github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
-github.com/stretchr/testify v1.7.0 h1:nwc3DEeHmmLAfoZucVR881uASk0Mfjw8xYJ99tb5CcY=
-github.com/stretchr/testify v1.7.0/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
+github.com/stretchr/objx v0.4.0/go.mod h1:YvHI0jy2hoMjB+UWwv71VJQ9isScKT/TqJzVSSt89Yw=
+github.com/stretchr/objx v0.5.0/go.mod h1:Yh+to48EsGEfYuaHDzXPcE3xhTkx73EhmCGUpEOglKo=
+github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
+github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
+github.com/stretchr/testify v1.8.1 h1:w7B6lhMri9wdJUVmEZPGGhZzrYTPvgJArz7wNPgYKsk=
+github.com/stretchr/testify v1.8.1/go.mod h1:w2LPCIKwWwSfY2zedu0+kehJoqGctiVI29o6fzry7u4=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
-gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b h1:h8qDotaEPuJATrMmW04NCwg7v22aHH28wwpauUhK9Oo=
-gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
+gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
+gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

main.go

@@ -53,7 +53,7 @@ func main() {
 func mainCmd(flags cmdFlags) error {
 	// If the version flag (-version) was given, print the version and exit.
 	if flags.version {
-		fmt.Println(version)
+		fmt.Println(version) //nolint:forbidigo
 		return nil
 	}
 

retry/retry.go

@@ -137,7 +137,8 @@ func maybeTimed(parent context.Context, timeout time.Duration) (context.Context,
 }
 
 // jitter returns a random duration in the range of:
-//   [0, variance)
+//
+//	[0, variance)
 func jitter(variance time.Duration) time.Duration {
 	if variance <= 0 {
 		return 0

retry/task.go

@@ -41,7 +41,7 @@ type HTTPTask struct {
 }
 
 func (t HTTPTask) Run(ctx context.Context) error {
-	request, err := http.NewRequest("GET", t.URL, nil)
+	request, err := http.NewRequest(http.MethodGet, t.URL, nil)
 	if err != nil {
 		return err
 	}