Apply suggestions from code review

Fix variable usage in message. Co-authored-by: Sascha Schwarze <schwarzs@de.ibm.com>
shipwright-io · Mar 11, 2024 · 8fc7dae · 8fc7dae
1 parent 121a33b
commit 8fc7dae
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/validate/strategies.go b/pkg/validate/strategies.go
@@ -69,7 +69,7 @@ func (s Strategy) validateBuildStrategy(ctx context.Context, strategyName string
 
 	if apierrors.IsNotFound(err) {
 		s.Build.Status.Reason = build.BuildReasonPtr(build.BuildStrategyNotFound)
-		s.Build.Status.Message = pointer.String(fmt.Sprintf("buildStrategy %s does not exist in namespace %s", s.Build.Spec.Strategy.Name, s.Build.Namespace))
+		s.Build.Status.Message = pointer.String(fmt.Sprintf("buildStrategy %s does not exist in namespace %s", strategyName, s.Build.Namespace))
 		return nil
 	}
 
@@ -87,7 +87,7 @@ func (s Strategy) validateClusterBuildStrategy(ctx context.Context, strategyName
 
 	if apierrors.IsNotFound(err) {
 		s.Build.Status.Reason = build.BuildReasonPtr(build.ClusterBuildStrategyNotFound)
-		s.Build.Status.Message = pointer.String(fmt.Sprintf("clusterBuildStrategy %s does not exist", s.Build.Spec.Strategy.Name))
+		s.Build.Status.Message = pointer.String(fmt.Sprintf("clusterBuildStrategy %s does not exist", strategyName))
 		return nil
 	}