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

Config Doc - Some modules miss the update to only trigger the annotation processor for main classes #42419

Closed
gsmet opened this issue Aug 8, 2024 · 1 comment · Fixed by #42443
Closed

Config Doc - Some modules miss the update to only trigger the annotation processor for main classes #42419

gsmet opened this issue Aug 8, 2024 · 1 comment · Fixed by #42443
Assignees
@gsmet
Labels
area/config
Milestone
3.14.0.CR1

Comments

@gsmet
Copy link
Member

gsmet commented Aug 8, 2024

ack --xml '\-Alegacy'

and look at the indentation to find the culprits.
@gsmet gsmet self-assigned this Aug 8, 2024
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Aug 8, 2024

/cc @radcortez (config)

gsmet added a commit to gsmet/quarkus that referenced this issue Aug 9, 2024
@gsmet
Only enable the extension annotation processor for default-compile
29f0ab6 
This is a followup of the previous PR as this change was already
partially implemented but my search/replace missed some of them.

This should be a lot more consistent (even if there are still a few
modules that are a bit odd).

Fixes quarkusio#42419
@gsmet gsmet mentioned this issue Aug 9, 2024
Merged
@quarkus-bot quarkus-bot bot added this to the 3.14 - main milestone Aug 11, 2024
frne pushed a commit to frne/quarkus that referenced this issue Aug 13, 2024
@gsmet @frne-trifork
Only enable the extension annotation processor for default-compile
9ede9de 
This is a followup of the previous PR as this change was already
partially implemented but my search/replace missed some of them.

This should be a lot more consistent (even if there are still a few
modules that are a bit odd).

Fixes quarkusio#42419
danielsoro pushed a commit to danielsoro/quarkus that referenced this issue Sep 20, 2024
@gsmet @danielsoro
Only enable the extension annotation processor for default-compile
73d980c 
This is a followup of the previous PR as this change was already
partially implemented but my search/replace missed some of them.

This should be a lot more consistent (even if there are still a few
modules that are a bit odd).

Fixes quarkusio#42419
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gsmet gsmet

Labels
area/config
Projects
None yet
Milestone
3.14.0.CR1
Development

Successfully merging a pull request may close this issue.

Only enable the extension annotation processor for default-compile gsmet/quarkus
1 participant
@gsmet