Fix Go linting (#451)

* Remove deactivated Go linters to fix build

* Fix Go linting by ignoring fmt.Fprintln errors

The reported fmt.Fprintln errors were causing the build to fail.
[It is standard practice not to check for these errors][1], so it is
fine to ignore them.

[1]: https://stackoverflow.com/a/77629756/23999321

* Update name of linter as old name is deprecated

.golangci.yml

@@ -29,24 +29,22 @@ linters:
     - revive
     - goprintffuncname
     - godox
-    - gomnd
     - gosec
     - gosimple
     - govet
     - ineffassign
     - lll
     - misspell
+    - mnd
     - prealloc
     - nakedret
     - rowserrcheck
     - exportloopref
     - staticcheck
-    - structcheck
     - stylecheck
     - typecheck
     - unconvert
     - unparam
     - unused
-    - varcheck
     - whitespace
     - wsl

internal/github/action.go

@@ -30,7 +30,7 @@ type Action struct {
 func (a *Action) CheckLabels(stdout, stderr io.Writer) int {
 	a.Stdout = stdout
 	a.Stderr = stderr
-	fmt.Fprintln(a.Stdout, "Checking GitHub labels ...")
+	fmt.Fprintln(a.Stdout, "Checking GitHub labels ...") //nolint:errcheck
 
 	pr := pullrequest.New(
 		a.repositoryOwner(),
@@ -46,7 +46,7 @@ func (a *Action) CheckLabels(stdout, stderr io.Writer) int {
 	a.runCheck(pr.Labels.HasAnyOf, a.anyRequired(), a.prefixMode())
 
 	if len(a.successMsg) > 0 {
-		fmt.Fprintln(a.Stdout, a.trimTrailingNewLine(a.successMsg))
+		fmt.Fprintln(a.Stdout, a.trimTrailingNewLine(a.successMsg)) //nolint:errcheck
 	}
 
 	if len(a.failMsg) > 0 {
@@ -63,7 +63,7 @@ func (a *Action) CheckLabels(stdout, stderr io.Writer) int {
 func (a *Action) handleFailure() int {
 	a.outputResult("failure")
 	err := errors.New(a.trimTrailingNewLine(a.failMsg))
-	fmt.Fprintln(a.Stderr, "::error::", err)
+	fmt.Fprintln(a.Stderr, "::error::", err) //nolint:errcheck
 
 	if a.allowFailure() {
 		return 0