gha: update to go 1.17, update golangci-lint to v1.44, and gofmt

.github/workflows/ci.yml

@@ -17,7 +17,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v2
       with:
-        go-version: 1.16.x
+        go-version: 1.17.x
       id: go
 
     - name: Setup Go binary path
@@ -39,12 +39,10 @@ jobs:
         path: src/github.com/containerd/project
 
     - name: Install dependencies
-      env:
-        GO111MODULE: off
       run: |
-        go get -u github.com/vbatts/git-validation
-        go get -u github.com/kunalkushwaha/ltag
-        GO111MODULE=on go get github.com/golangci/golangci-lint/cmd/golangci-lint@v1.23.8
+        go install github.com/vbatts/git-validation@latest
+        go install github.com/kunalkushwaha/ltag@latest
+        go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.44.0
 
     - name: Check DCO/whitespace/commit message
       env:

.golangci.yml

@@ -1,16 +1,16 @@
 linters:
   enable:
-    - structcheck
-    - varcheck
-    - staticcheck
-    - unconvert
     - gofmt
     - goimports
-    - golint
     - ineffassign
-    - vet
-    - unused
     - misspell
+    - revive
+    - staticcheck
+    - structcheck
+    - unconvert
+    - unused
+    - varcheck
+    - vet
   disable:
     - errcheck
 

console.go

@@ -78,7 +78,7 @@ func Current() (c Console) {
 }
 
 // ConsoleFromFile returns a console using the provided file
-// nolint:golint
+// nolint:revive
 func ConsoleFromFile(f File) (Console, error) {
 	if err := checkConsole(f); err != nil {
 		return nil, err

console_linux.go

@@ -1,3 +1,4 @@
+//go:build linux
 // +build linux
 
 /*

console_test.go

@@ -1,3 +1,4 @@
+//go:build linux || solaris || zos || freebsd
 // +build linux solaris zos freebsd
 
 /*

console_unix.go

@@ -1,3 +1,4 @@
+//go:build darwin || freebsd || linux || netbsd || openbsd || solaris
 // +build darwin freebsd linux netbsd openbsd solaris
 
 /*

console_zos.go

@@ -1,3 +1,4 @@
+//go:build zos
 // +build zos
 
 /*
@@ -32,7 +33,7 @@ func NewPty() (Console, string, error) {
 	var f File
 	var err error
 	var slave string
-	for i := 0;; i++ {
+	for i := 0; ; i++ {
 		ptyp := fmt.Sprintf("/dev/ptyp%04d", i)
 		f, err = os.OpenFile(ptyp, os.O_RDWR, 0600)
 		if err == nil {

pty_freebsd_cgo.go

@@ -1,3 +1,4 @@
+//go:build freebsd && cgo
 // +build freebsd,cgo
 
 /*

pty_freebsd_nocgo.go

@@ -1,3 +1,4 @@
+//go:build freebsd && !cgo
 // +build freebsd,!cgo
 
 /*

pty_unix.go

@@ -1,3 +1,4 @@
+//go:build darwin || linux || netbsd || openbsd || solaris
 // +build darwin linux netbsd openbsd solaris
 
 /*

tc_freebsd_cgo.go

@@ -1,3 +1,4 @@
+//go:build freebsd && cgo
 // +build freebsd,cgo
 
 /*

tc_freebsd_nocgo.go

@@ -1,3 +1,4 @@
+//go:build freebsd && !cgo
 // +build freebsd,!cgo
 
 /*

tc_openbsd_cgo.go

@@ -1,3 +1,4 @@
+//go:build openbsd && cgo
 // +build openbsd,cgo
 
 /*

tc_openbsd_nocgo.go

@@ -1,3 +1,4 @@
+//go:build openbsd && !cgo
 // +build openbsd,!cgo
 
 /*

tc_solaris_cgo.go

@@ -1,3 +1,4 @@
+//go:build solaris && cgo
 // +build solaris,cgo
 
 /*

tc_solaris_nocgo.go

@@ -1,3 +1,4 @@
+//go:build solaris && !cgo
 // +build solaris,!cgo
 
 /*

tc_unix.go

@@ -1,3 +1,4 @@
+//go:build darwin || freebsd || linux || netbsd || openbsd || solaris || zos
 // +build darwin freebsd linux netbsd openbsd solaris zos
 
 /*