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
ArC: allow putting bean enablement annotations on stereotypes #37302
Conversation
I just realized I forgot to add something to the documentation. Will fix next week. EDIT: done. |
This comment has been minimized.
This comment has been minimized.
0a93546
to
7766662
Compare
I removed the support for custom stereotypes from this PR, which likely isn't a big deal, and it avoids a build step cycle. |
🙈 The PR is closed and the preview is expired. |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small comment, otherwise looks good.
Also, thumbs up for good test coverage!
...nsions/arc/deployment/src/main/java/io/quarkus/arc/deployment/BuildTimeEnabledProcessor.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the new usage of EquivalenceKey
! And the tests are nice too ofc.
...nsions/arc/deployment/src/main/java/io/quarkus/arc/deployment/BuildTimeEnabledProcessor.java
Outdated
Show resolved
Hide resolved
...nsions/arc/deployment/src/main/java/io/quarkus/arc/deployment/BuildTimeEnabledProcessor.java
Outdated
Show resolved
Hide resolved
...nsions/arc/deployment/src/main/java/io/quarkus/arc/deployment/BuildTimeEnabledProcessor.java
Outdated
Show resolved
Hide resolved
It would probably be best to write this algorithm in a lazy fashion (driven by the annotation transformation demands), but that would require breaking an extension API (specifically, it wouldn't be possible to produce `BuildTimeConditionBuildItem`). Hence, this commit enhances the eager algorithm for bean enablement scanning to also scan stereotypes, relying on subclass information in the Jandex index to support `@Inherited` stereotypes.
7766662
to
c197f41
Compare
@mkouba comments addressed. |
✔️ The latest workflow run for the pull request has completed successfully. It should be safe to merge provided you have a look at the other checks in the summary. |
It would probably be best to write this algorithm in a lazy fashion (driven by the annotation transformation demands), but that would require breaking an extension API (specifically, it wouldn't be possible to produce
BuildTimeConditionBuildItem
). Hence, this commit enhances the eager algorithm for bean enablement scanning to also scan stereotypes, relying on subclass information in the Jandex index to support@Inherited
stereotypes.Fixes #37117