Skip to content

Commit

Permalink
comments
Browse files Browse the repository at this point in the history
Signed-off-by: Future-Outlier <[email protected]>
  • Loading branch information
Future-Outlier committed Nov 15, 2024
1 parent b224a02 commit 7ed9be2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions flytepropeller/pkg/compiler/validators/typing.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,10 @@ func isSuperTypeInJSON(sourceMetaData, targetMetaData *structpb.Struct) bool {
// Compare the two schemas
errs := jscmp.Compare(tgtSchema, srcSchema)

// json-schema-compare does not support additionalProperties
// If len(errs) >= 2, then the schemas are not compatible
// json-schema-compare does not support additionalProperties=true
if len(errs) == 1 {
return strings.Contains(errs[0].Error(), "FIXME additionalProperties not implemented")
return strings.Contains(errs[0].Error(), "additionalProperties not implemented")
}

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

View check run for this annotation

Codecov / codecov/patch

flytepropeller/pkg/compiler/validators/typing.go#L48-L49

Added lines #L48 - L49 were not covered by tests

return len(errs) == 0
Expand Down

0 comments on commit 7ed9be2

Please sign in to comment.