Skip to content

fix: based on suggested change

Codecov / codecov/patch succeeded Dec 28, 2024 in 1s

58.20% of diff hit (target 37.10%)

View this Pull Request on Codecov

58.20% of diff hit (target 37.10%)

Annotations

Check warning on line 241 in flyteidl/gen/pb-go/flyteidl/plugins/spark.pb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/gen/pb-go/flyteidl/plugins/spark.pb.go#L237-L241

Added lines #L237 - L241 were not covered by tests

Check warning on line 248 in flyteidl/gen/pb-go/flyteidl/plugins/spark.pb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/gen/pb-go/flyteidl/plugins/spark.pb.go#L244-L248

Added lines #L244 - L248 were not covered by tests

Check warning on line 287 in flyteplugins/go/tasks/pluginmachinery/flytek8s/pod_helper.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/flytek8s/pod_helper.go#L286-L287

Added lines #L286 - L287 were not covered by tests

Check warning on line 209 in flyteplugins/go/tasks/plugins/k8s/spark/spark.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/k8s/spark/spark.go#L207-L209

Added lines #L207 - L209 were not covered by tests

Check warning on line 214 in flyteplugins/go/tasks/plugins/k8s/spark/spark.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/k8s/spark/spark.go#L213-L214

Added lines #L213 - L214 were not covered by tests

Check warning on line 256 in flyteplugins/go/tasks/plugins/k8s/spark/spark.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/k8s/spark/spark.go#L254-L256

Added lines #L254 - L256 were not covered by tests

Check warning on line 261 in flyteplugins/go/tasks/plugins/k8s/spark/spark.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/k8s/spark/spark.go#L260-L261

Added lines #L260 - L261 were not covered by tests