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.

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

Document that @DefaultValue can be used on a record component #30460

Closed
wilkinsona opened this issue Mar 29, 2022 · 0 comments
Closed

Document that @DefaultValue can be used on a record component #30460

wilkinsona opened this issue Mar 29, 2022 · 0 comments
Assignees
Labels
type: documentation
Milestone

Comments

@wilkinsona
Copy link
Member

@wilkinsona wilkinsona commented Mar 29, 2022

With thanks to @danvega for bringing this to our attention, we made a mistake in #29010.

An annotation can be used on a record component if it targets one or more of FIELD, METHOD, PARAMETER, and RECORD_COMPONENT. The annotation is then propagated to the record component's field, accessor method, constructor parameter, or the record component itself based on these targets. @DefaultValue already has a target of PARAMETER so prior to the changes in #29010 it could be used on a record component and would then be propagated to the record's constructor. From there it would be found by configuration property binding and the annotation processor.

We can update the documentation and backport some tests in 2.5 as things already work there without any changes to main code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation
Projects
None yet
Development

No branches or pull requests

1 participant