fix(lint): suppress unparam warning on applySaturationDecisions

jia-gao · jia-gao · commit bbdb5f90e924 · 2026-04-14T15:14:30.000-07:00
The error return is always nil today because all errors are logged and
handled inline. Keep the signature for future use when partial failures
(e.g. some VAs succeed, others fail) are propagated to the caller.

Signed-off-by: Jiazhou Gao &lt;gjz140103@gmail.com&gt;
diff --git a/internal/engines/saturation/engine.go b/internal/engines/saturation/engine.go
@@ -884,6 +884,8 @@ func (e *Engine) RunSaturationAnalysis(
 }
 
 // applySaturationDecisions updates VA status and emits metrics based on Saturation decisions.
+//
+//nolint:unparam // error return kept for future use when partial failures are propagated
 func (e *Engine) applySaturationDecisions(
 	ctx context.Context,
 	decisions []interfaces.VariantDecision,

Original file line number	Diff line number	Diff line change
`@@ -884,6 +884,8 @@ func (e *Engine) RunSaturationAnalysis(`
`884`	`884`	`}`
`885`	`885`
`886`	`886`	`// applySaturationDecisions updates VA status and emits metrics based on Saturation decisions.`
	`887`	`+//`
	`888`	`+//nolint:unparam // error return kept for future use when partial failures are propagated`
`887`	`889`	`func (e *Engine) applySaturationDecisions(`
`888`	`890`	`ctx context.Context,`
`889`	`891`	`decisions []interfaces.VariantDecision,`