Omit nil check; len() for nil slices is defined as zero, reported by linter

tabito-hara · tabito-hara · commit 008508f55c14 · 2025-09-01T22:47:34.000+09:00
diff --git a/internal/service/synthetics/canary.go b/internal/service/synthetics/canary.go
@@ -394,7 +394,7 @@ func resourceCanaryRead(ctx context.Context, d *schema.ResourceData, meta any) d
 	}.String()
 	d.Set(names.AttrARN, canaryArn)
 	d.Set("artifact_s3_location", canary.ArtifactS3Location)
-	if canary.EngineConfigs != nil && len(canary.EngineConfigs) > 0 {
+	if len(canary.EngineConfigs) > 0 {
 		d.Set("engine_arn", canary.EngineConfigs[0].EngineArn)
 	}
 	d.Set(names.AttrExecutionRoleARN, canary.ExecutionRoleArn)

Original file line number	Diff line number	Diff line change
`@@ -394,7 +394,7 @@ func resourceCanaryRead(ctx context.Context, d *schema.ResourceData, meta any) d`
`394`	`394`	`}.String()`
`395`	`395`	`d.Set(names.AttrARN, canaryArn)`
`396`	`396`	`d.Set("artifact_s3_location", canary.ArtifactS3Location)`
`397`		`- if canary.EngineConfigs != nil && len(canary.EngineConfigs) > 0 {`
	`397`	`+ if len(canary.EngineConfigs) > 0 {`
`398`	`398`	`d.Set("engine_arn", canary.EngineConfigs[0].EngineArn)`
`399`	`399`	`}`
`400`	`400`	`d.Set(names.AttrExecutionRoleARN, canary.ExecutionRoleArn)`