Skip to content

extension API improvements#536

Merged
Ladicek merged 2 commits intojakartaee:masterfrom
Ladicek:api-improvements
Oct 5, 2021
Merged

extension API improvements#536
Ladicek merged 2 commits intojakartaee:masterfrom
Ladicek:api-improvements

Conversation

@Ladicek
Copy link
Copy Markdown
Member

@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 Ladicek requested a review from graemerocher October 5, 2021 12:33
@Ladicek
Copy link
Copy Markdown
Member 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.

2 participants