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

Remove Enhancement.BeanDefiningAnnotations #569

Merged
merged 1 commit into from
Dec 2, 2021

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Dec 1, 2021

The marker annotation @Enhancement.BeanDefiningAnnotations
used to represent all bean defining annotations for the purpose
of restricting the set of types on which @Enhancement should
be performed. That, however, is pretty useless: @Enhancement
is only performed on types that were discovered during type
discovery, and all such types have a bean defining annotation.
(Either they have it directly, or @Dependent is implied
in case of classes added through @Discovery that don't have
a bean defining annotation.)

The marker annotation `@Enhancement.BeanDefiningAnnotations`
used to represent all bean defining annotations for the purpose
of restricting the set of types on which `@Enhancement` should
be performed. That, however, is pretty useless: `@Enhancement`
is only performed on types that were discovered during type
discovery, and all such types have a bean defining annotation.
(Either they have it directly, or `@Dependent` is implied
in case of classes added through `@Discovery` that don't have
a bean defining annotation.)
@Ladicek Ladicek added lite-extension-api Issues related to CDI Lite extension API proposal Lite Related to CDI Lite labels Dec 1, 2021
@Ladicek Ladicek merged commit c531b40 into jakartaee:master Dec 2, 2021
@Ladicek Ladicek deleted the remove-enhancement-bdas branch December 2, 2021 10:06
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.

3 participants