fix(ci): update ignore directives for staticcheck

Switch golangci-lint nolint:unused to staticcheck lint:ignore style Signed-off-by: Dominic Evans <[email protected]>
IBM · Feb 3, 2025 · b3aef99 · b3aef99
1 parent b32d5fc
commit b3aef99
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/balance_strategy.go b/balance_strategy.go
@@ -989,7 +989,7 @@ func (p *partitionMovements) getTheActualPartitionToBeMoved(partition topicParti
 	return reversePairPartition
 }
 
-//nolint:unused // this is used but only in unittests as a helper (which are excluded by the integration build tag)
+//lint:ignore U1000 // this is used but only in unittests as a helper (which are excluded by the integration build tag)
 func (p *partitionMovements) isLinked(src, dst string, pairs []consumerPair, currentPath []string) ([]string, bool) {
 	if src == dst {
 		return currentPath, false
@@ -1024,7 +1024,7 @@ func (p *partitionMovements) isLinked(src, dst string, pairs []consumerPair, cur
 	return currentPath, false
 }
 
-//nolint:unused // this is used but only in unittests as a helper (which are excluded by the integration build tag)
+//lint:ignore U1000 // this is used but only in unittests as a helper (which are excluded by the integration build tag)
 func (p *partitionMovements) in(cycle []string, cycles [][]string) bool {
 	superCycle := make([]string, len(cycle)-1)
 	for i := 0; i < len(cycle)-1; i++ {
@@ -1039,7 +1039,7 @@ func (p *partitionMovements) in(cycle []string, cycles [][]string) bool {
 	return false
 }
 
-//nolint:unused // this is used but only in unittests as a helper (which are excluded by the integration build tag)
+//lint:ignore U1000 // this is used but only in unittests as a helper (which are excluded by the integration build tag)
 func (p *partitionMovements) hasCycles(pairs []consumerPair) bool {
 	cycles := make([][]string, 0)
 	for _, pair := range pairs {
@@ -1071,7 +1071,7 @@ func (p *partitionMovements) hasCycles(pairs []consumerPair) bool {
 	return false
 }
 
-//nolint:unused // this is used but only in unittests as a helper (which are excluded by the integration build tag)
+//lint:ignore U1000 // this is used but only in unittests as a helper (which are excluded by the integration build tag)
 func (p *partitionMovements) isSticky() bool {
 	for topic, movements := range p.PartitionMovementsByTopic {
 		movementPairs := make([]consumerPair, len(movements))
@@ -1089,7 +1089,7 @@ func (p *partitionMovements) isSticky() bool {
 	return true
 }
 
-//nolint:unused // this is used but only in unittests as a helper (which are excluded by the integration build tag)
+//lint:ignore U1000 // this is used but only in unittests as a helper (which are excluded by the integration build tag)
 func indexOfSubList(source []string, target []string) int {
 	targetSize := len(target)
 	maxCandidate := len(source) - targetSize

diff --git a/metrics_helpers_test.go b/metrics_helpers_test.go
@@ -116,7 +116,7 @@ func minCountHistogramValidator(name string, minCount int) *metricValidator {
 	})
 }
 
-//nolint:unused // this is used but only in unittests which are excluded by the integration build tag
+//lint:ignore U1000 // this is used but only in unittests which are excluded by the integration build tag
 func minMaxHistogramValidator(name string, expectedMin int, expectedMax int) *metricValidator {
 	return histogramValidator(name, func(t *testing.T, histogram metrics.Histogram) {
 		t.Helper()