Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use jsonpb AllowUnknownFields everywhere #5521

Merged
merged 3 commits into from
Jul 4, 2024

make lint-fix

a4aec9c
Select commit
Loading
Failed to load commit list.
Merged

Use jsonpb AllowUnknownFields everywhere #5521

make lint-fix
a4aec9c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jun 28, 2024 in 1s

58.82% of diff hit (target 60.99%)

View this Pull Request on Codecov

58.82% of diff hit (target 60.99%)

Annotations

Check warning on line 58 in flytectl/cmd/get/node_execution.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/cmd/get/node_execution.go#L58 

Added line #L58 was not covered by tests

Check warning on line 73 in flytectl/cmd/get/node_execution.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytectl/cmd/get/node_execution.go#L73 

Added line #L73 was not covered by tests

Check warning on line 25 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go#L25 

Added line #L25 was not covered by tests

Check warning on line 30 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go#L30 

Added line #L30 was not covered by tests

Check warning on line 38 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go#L38 

Added line #L38 was not covered by tests

Check warning on line 43 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go#L43 

Added line #L43 was not covered by tests

Check warning on line 51 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go#L51 

Added line #L51 was not covered by tests

Check warning on line 56 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/gate.go#L56 

Added line #L56 was not covered by tests

Check warning on line 13 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/identifier.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/identifier.go#L12-L13 

Added lines #L12 - L13 were not covered by tests

Check warning on line 18 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/identifier.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/identifier.go#L16-L18 

Added lines #L16 - L18 were not covered by tests

Check warning on line 18 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go#L18 

Added line #L18 was not covered by tests

Check warning on line 23 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go#L23 

Added line #L23 was not covered by tests

Check warning on line 88 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go#L88 

Added line #L88 was not covered by tests

Check warning on line 93 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/nodes.go#L93 

Added line #L93 was not covered by tests

Check warning on line 27 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/tasks.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/tasks.go#L27 

Added line #L27 was not covered by tests

Check warning on line 197 in flytepropeller/pkg/apis/flyteworkflow/v1alpha1/workflow.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/apis/flyteworkflow/v1alpha1/workflow.go#L197 

Added line #L197 was not covered by tests
View more details on Codecov