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

[Fix] Add logger for compiler and marshal while comparing union #6034

Merged
merged 34 commits into from
Nov 20, 2024

better error msg format

1f92e5c
Select commit
Loading
Failed to load commit list.
Merged

[Fix] Add logger for compiler and marshal while comparing union #6034

better error msg format
1f92e5c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 20, 2024 in 1s

19.23% of diff hit (target 37.07%)

View this Pull Request on Codecov

19.23% of diff hit (target 37.07%)

Annotations

Check warning on line 35 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L33-L35 

Added lines #L33 - L35 were not covered by tests

Check warning on line 40 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L38-L40 

Added lines #L38 - L40 were not covered by tests

Check warning on line 46 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L46 

Added line #L46 was not covered by tests

Check warning on line 51 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L51 

Added line #L51 was not covered by tests

Check warning on line 59 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L57-L59 

Added lines #L57 - L59 were not covered by tests

Check warning on line 64 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L62-L64 

Added lines #L62 - L64 were not covered by tests

Check warning on line 84 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L84 

Added line #L84 was not covered by tests

Check warning on line 90 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L90 

Added line #L90 was not covered by tests

Check warning on line 97 in flytepropeller/pkg/compiler/validators/typing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L97 

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