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

extension API improvements #536

Merged
merged 2 commits into from Oct 5, 2021
Merged

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Oct 5, 2021

No description provided.

- modified the `@Processing` phase to also execute for synthetic beans
  and observers, after `@Synthesis` is finished
- added annotation transformations for method parameters (`ParameterConfig`)
- changed `ArrayType` to use a component type representation,
  instead of a representation with element type + number of dimensions
- clarified what happens with non-existing annotation members defined
  using `AnnotationBuilder` in the resulting `AnnotationInfo`
- clarified the set of error conditions for `Types.ofArray()`
- settled on the current design for `Parameters`, removed the comment
  that outlines another possible design
- removed some TODOs where alternative design options exist, but they
  have no obvious advantage (one design simply has to be chosen)
@Ladicek
Copy link
Contributor Author

Ladicek commented Oct 5, 2021

As a separate commit, I added support for records in the language model, because I thought perhaps we should stay current :-) If that is problematic for some reason, I'd be fine with removing it.

@Ladicek Ladicek merged commit 5f55bfe into jakartaee:master Oct 5, 2021
@Ladicek Ladicek deleted the api-improvements branch October 5, 2021 14:06
@Ladicek Ladicek added Lite Related to CDI Lite lite-extension-api Issues related to CDI Lite extension API proposal labels Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Lite Related to CDI Lite lite-extension-api Issues related to CDI Lite extension API proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants