Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@laeubi
Copy link
Contributor

@laeubi laeubi commented Nov 9, 2024

Fix #646

@stbischof
Copy link
Contributor

Could you please add a reference to the Lines in Spec -text

@laeubi
Copy link
Contributor Author

laeubi commented Nov 20, 2024

Could you please add a reference to the Lines in Spec -text

See here

If the method type is Class or Class[], then the property type must be String or String[], respectively, whose values are fully qualified class names in the form returned by the Class.getName() method.

It is not specifically explained how empty arrays must be handled, but as there is no way to define a "non-property" (it would still have the empty String as value) it is valid to assume that the case of an empty array should result in no property defined at all.

@stbischof
Copy link
Contributor

@bjhargrave any opinion on that?

@bjhargrave
Copy link
Member

@bjhargrave any opinion on that?

The TCK failed. So that should be fixed first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ds annotation tck] Usage of empty array and classes in componet property types is not covered by the TCK

3 participants