Skip to content

Commit

Permalink
add nolint wsl to questionable finding
Browse files Browse the repository at this point in the history
  • Loading branch information
druppelt committed Sep 11, 2024
1 parent a9e13fc commit 6aeb746
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/calc/deployment.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ func deployment(deployment appsv1.Deployment) (*ResourceUsage, error) {
if podOverheadInt > math.MaxInt32 || podOverheadInt < math.MinInt32 {
return nil, fmt.Errorf("deployment: %s maxSurge - maxUnavailable (%d-%d) was out of bounds for int32", deployment.Name, maxSurge, maxUnavailable)
}
podOverhead = int32(podOverheadInt) //nolint:gosec
podOverhead = int32(podOverheadInt) //nolint:gosec,wsl

Check failure on line 91 in internal/calc/deployment.go

View workflow job for this annotation

GitHub Actions / lint

directive `//nolint:gosec,wsl` should provide explanation such as `//nolint:gosec,wsl // this is why` (nolintlint)

resourceOverhead = (float64(podOverhead) / float64(*replicas)) + 1
default:
Expand Down

0 comments on commit 6aeb746

Please sign in to comment.