FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Gradle 9.0.0 by catilac · Pull Request #1459 · processing/processing4 · GitHub

Gradle 9.0.0 - #1459

Closed
catilac wants to merge 3 commits into
processing:mainfrom
catilac:gradle-9.0.0
Closed

Gradle 9.0.0#1459
catilac wants to merge 3 commits into
processing:mainfrom
catilac:gradle-9.0.0

Conversation

catilac commented Mar 3, 2026

Copy link
Copy Markdown
Collaborator

For some reason github actions is now using Gradle 9.0.0. Unfortunately I missed what exactly changed in that regard.
Locally we were using an earlier version. In this PR I upgraded to Gradle 9.0.0 and have attempted to make it work.

NOTE: I did end up using a combination of Claude code and internet searches to figure this one out.

catilac requested a review from tychedelia March 3, 2026 00:01
tasks.register("writeVersion") {
// make the version available to the plugin at runtime by writing it to a properties file in the resources directory
doLast {
val version = project.version.toString().takeIf { it != "unspecified" } ?: return@doLast

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This will mean that the tests will run w core 4.3.4 instead of the locally build one

catilac marked this pull request as draft March 10, 2026 13:55
marksherman mentioned this pull request Mar 16, 2026

Copy link
Copy Markdown

Confirming this does run tests, build, etc successfully for me locally (MacOS)

Copy link
Copy Markdown
Member

As mentioned before on discord, unfortunately that is because it skips the failing test rather than solve it

catilac closed this Apr 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL