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
Stop index warn by excluding resource removed by extension from indexing #28214
Stop index warn by excluding resource removed by extension from indexing #28214
Conversation
cc @geoand I can divide this to 2 PRs if you think it's appropriate. I added this as one PR as changes are tiny. |
This comment has been minimized.
This comment has been minimized.
Hi, Thanks for this! Furthermore, I'd like @aloubyansky to verify this |
Looks good, thanks! |
18b106f
to
6fd688c
Compare
Excluded resources removed by extension from indexing as according to io.quarkus.maven.ExtensionDescriptorMojo#removedResources should be equivalent of quarkus.class-loading.removed-resources, however later classes are excluded from indexing while former are not.
6fd688c
to
34b21a7
Compare
PR is split into 2 commits. Thank you for reviews. |
✖ This workflow run has failed but no jobs reported an error. Something weird happened, please check the workflow run page carefully: it might be an issue with the workflow configuration itself. |
fixes: #27907
Excludes resources removed by extension from indexing as according to
io.quarkus.maven.ExtensionDescriptorMojo#removedResources
should be equivalent ofquarkus.class-loading.removed-resources
, however later classes are excluded from indexing while former are not. Fix is used to excludeorg.keycloak.admin.client.JacksonProvider
that extendsorg.jboss.resteasy.plugins.providers.jackson.ResteasyJackson2Provider
from indexing. The file is never used as we setio.quarkus.keycloak.admin.client.reactive.runtime.ResteasyReactiveKeycloakAdminClientRecorder#setClientProvider
and it produces incorrect warning (see linked issue).