Enable nolintlint linter

.golangci.yml

@@ -18,6 +18,7 @@ linters:
     - goheader      # Checks is file headers matche a given pattern
     - intrange      # Checking for loops that could use an integer range
     - perfsprint    # Checks for faster fmt.Sprintf alternatives
+    - nolintlint    # Find ill-formed or insufficiently explained nolint directives
     - revive        # Stricter drop-in replacement for golint
     - testifylint   # Checks usage of github.com/stretchr/testify
     - unconvert     # Checks for unnecessary type conversions

internal/pkg/sysinfo/probes/linux/linux.go

@@ -133,7 +133,7 @@ func newUnameProber() unameProber {
 }
 
 func parseUname(utsname *syscall.Utsname) *uname {
-	convert := func(chars utsStringPtr) unameField {
+	convert := func(chars *[65]utsChar) unameField {
 		var buf [65]byte
 		var i int
 		for pos, ch := range *chars {

internal/pkg/sysinfo/probes/linux/types.go

@@ -18,6 +18,4 @@ limitations under the License.
 
 package linux
 
-func utsChar(ch rune) int8 { return int8(ch) } //nolint:deadcode,unused // just for tests 🙄
-
-type utsStringPtr *[65]int8
+type utsChar = int8

internal/pkg/sysinfo/probes/linux/types_arm.go

@@ -18,6 +18,4 @@ limitations under the License.
 
 package linux
 
-func utsChar(ch rune) uint8 { return uint8(ch) } //nolint:deadcode,unused // just for tests 🙄
-
-type utsStringPtr *[65]uint8
+type utsChar = uint8

pkg/autopilot/controller/root/root.go

@@ -19,7 +19,6 @@ import (
 )
 
 // TODO: decide on renaming root.RootConfig -> root.Config
-// nolint:revive
 type RootConfig struct {
 	InvocationID        string
 	KubeConfig          string

pkg/component/controller/workerconfig/reconciler_test.go

@@ -584,7 +584,7 @@ func TestReconciler_runReconcileLoop(t *testing.T) {
 
 	underTest.runReconcileLoop(ctx, updates, nil)
 
-	switch ctx.Err() { //nolint:errorlint // as per context contract
+	switch ctx.Err() {
 	case context.Canceled:
 		break // this is the good case
 	case context.DeadlineExceeded: