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

Fixed deprecation warnings caused by QuarkusPlugin #38249

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

ribafish
Copy link
Contributor

@ribafish ribafish commented Jan 17, 2024

Fixed deprecation warnings for Gradle 9 caused by QuarkusPlugin. Made sure the sourceSets are created before accessing their configurations.

Explanation:

Creating configurations that a sourceSet creates is deprecated, resulting in this deprecation warning. Doing it this way will also cause problems when configuration permitted usages are misaligned, resulting in a second deprecation warning.

With the fix applied, custom sourcesets (and their configurations) are created first, then the configurations are configured, after which sourcesets are configured as part of registerTasks method, which also removes the deprecation warnings.

Made sure the sourceSets are created before accessing their configurations.
@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle labels Jan 17, 2024
@ribafish ribafish force-pushed the gk/quarkusPluginGradle9Fix branch from a696e0d to 4db15c3 Compare January 17, 2024 13:47
@ribafish ribafish changed the title Fixed deprecation warnings in QuarkusPlugin Fixed deprecation warnings caused by QuarkusPlugin Jan 17, 2024
@gsmet gsmet requested review from glefloch and aloubyansky January 17, 2024 15:21
Copy link

quarkus-bot bot commented Jan 17, 2024

✔️ 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.

You can consult the Develocity build scans.

@aloubyansky aloubyansky merged commit c7ad2cf into quarkusio:main Jan 17, 2024
21 of 34 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.9 - main milestone Jan 17, 2024
@ribafish ribafish deleted the gk/quarkusPluginGradle9Fix branch January 18, 2024 16:05
@gsmet gsmet modified the milestones: 3.9 - main, 3.7.0 Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants