paramTypeCombine: func(h int, m int, s int) string could be replaced with func(h, m, s int) string (gocritic)

paramTypeCombine: func(id string, address string) *networkv1alpha1.KnownCluster could be replaced with func(id, address string) *networkv1alpha1.KnownCluster (gocritic)

commentedOutCode: may want to remove commented-out code (gocritic)

rangeValCopy: each iteration copies 312 bytes (consider pointers or indexing) (gocritic)

G601: Implicit memory aliasing in for loop. (gosec)

issues found

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4, golangci/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

exported: comment on exported method KnownCluster.UpdateStatus should be of the form "UpdateStatus ..." (revive)

exported: comment on exported function ForgeKnownClusterName should be of the form "ForgeKnownClusterName ..." (revive)

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)