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

Avoid use of convention properties #4400

Merged

Conversation

kwvanderlinde
Copy link
Collaborator

@kwvanderlinde kwvanderlinde commented Nov 15, 2023

Identify the Bug or Feature request

Address #4398

Description of the Change

This changes avoids warning around convention properties and implicit test dependencies, and also upgrades to Gradle 8.4.

Also included is a refactoring to keep the list of java args in a common place so the jpackage and application task configurations don't have to be manually kept in sync.

Possible Drawbacks

Should be none.

Documentation Notes

N/A

Release Notes

N/A


This change is Reviewable

This addresses warnings around the `java` and `application` plugins removing their convention properties in Gradle 9.0.
This avoids a warning about `:test` compatibility with Gradle 9.0.
This brings proper Java 20 support instead of the partial support present in 8.2.1.
@kwvanderlinde kwvanderlinde force-pushed the refactor/4398-gradle-9-compatibility branch from befbb3b to 1d9b014 Compare November 15, 2023 20:30
@cwisniew cwisniew added the refactor Refactoring the code for optimal awesomeness. label Nov 15, 2023
@cwisniew cwisniew added this pull request to the merge queue Nov 15, 2023
Merged via the queue into RPTools:develop with commit eea2f24 Nov 15, 2023
4 checks passed
@kwvanderlinde kwvanderlinde deleted the refactor/4398-gradle-9-compatibility branch November 16, 2023 00:54
@cwisniew cwisniew added the code-maintenance Adding/editing javadocs, unit tests, formatting. label Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-maintenance Adding/editing javadocs, unit tests, formatting. refactor Refactoring the code for optimal awesomeness.
Projects
Development

Successfully merging this pull request may close these issues.

3 participants