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.

Sign up for GitHub

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

Jump to bottom

add a builder for AnnotationInstance #208

Closed
Ladicek opened this issue May 27, 2022 · 1 comment · Fixed by #209
Closed

add a builder for AnnotationInstance #208

Ladicek opened this issue May 27, 2022 · 1 comment · Fixed by #209
Assignees
@Ladicek
Milestone
3.0.0

Comments

@Ladicek
Copy link
Contributor

Ladicek commented May 27, 2022

Constructing AnnotationInstance manually is pretty ugly. Historically, that wasn't important, but Quarkus extensions (especially annotation transformations) do that pretty often. We could/should add a builder API that would make it much nicer.
@Ladicek Ladicek self-assigned this May 27, 2022
@Ladicek Ladicek added this to the 3.0.0 milestone May 27, 2022
@Ladicek Ladicek mentioned this issue May 30, 2022
Merged
@Ladicek Ladicek linked a pull request May 30, 2022 that will close this issue
add AnnotationInstance.builder() #209
Merged
@Ladicek
Copy link
Contributor Author

Ladicek commented May 30, 2022

Done in #209.

@Ladicek Ladicek closed this as completed May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ladicek Ladicek

Labels
None yet
Projects
None yet
Milestone
3.0.0
Development

Successfully merging a pull request may close this issue.

add AnnotationInstance.builder() Ladicek/jandex
1 participant
@Ladicek