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

Fix for ServiceLoaderFeature #11402

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

andriy-dmytruk
Copy link
Contributor

@andriy-dmytruk andriy-dmytruk commented Dec 3, 2024

ServiceLoaderFeature did not include all the @Requires annotations, as some were added with the @Requirements. I included such in the native-image beforeAnalysis. As an example, this fixes the usage of condition in io.micronaut.configuration.metrics.binder.web.ClientRequestMetricRegistryFilter.

@graemerocher graemerocher changed the base branch from 4.8.x to 4.7.x December 3, 2024 21:22
@graemerocher graemerocher changed the base branch from 4.7.x to 4.8.x December 3, 2024 21:22
@graemerocher
Copy link
Contributor

@andriy-dmytruk could you target 4.7.x?

ServiceLoaderFeature did not include all the @requires annotations, as some were added with the @requirements. I included such in the native-image beforeAnalysis. As an example, this fixes the usage of condition in io.micronaut.configuration.metrics.binder.web.ClientRequestMetricRegistryFilter.
@andriy-dmytruk andriy-dmytruk force-pushed the native-image-condition-fix branch from 1baf942 to d00b58e Compare December 3, 2024 21:32
@andriy-dmytruk andriy-dmytruk changed the base branch from 4.8.x to 4.7.x December 3, 2024 21:33
@graemerocher graemerocher merged commit a1d34fa into 4.7.x Dec 3, 2024
16 checks passed
@graemerocher graemerocher deleted the native-image-condition-fix branch December 3, 2024 21:41
@graemerocher graemerocher added type: bug Something isn't working relates-to: graal labels Dec 3, 2024
Copy link

sonarqubecloud bot commented Dec 3, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
67.6% Coverage on New Code (required ≥ 70%)
8 New Bugs (required ≤ 0)
3 New Blocker Issues (required ≤ 0)
5 New Critical Issues (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relates-to: graal type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants