Skip to content

Update Gradle Wrapper from 8.14.3 to 9.3.1 & also run tests with JDK 25#1550

Open
hankem wants to merge 4 commits intomainfrom
gradle-9
Open

Update Gradle Wrapper from 8.14.3 to 9.3.1 & also run tests with JDK 25#1550
hankem wants to merge 4 commits intomainfrom
gradle-9

Conversation

@hankem
Copy link
Member

@hankem hankem commented Nov 10, 2025

@hankem hankem changed the title Update Gradle Wrapper from 8.14.3 to 9.2 Update Gradle Wrapper from 8.14.3 to 9.2.0 Nov 10, 2025
@hankem hankem changed the title Update Gradle Wrapper from 8.14.3 to 9.2.0 Update Gradle Wrapper from 8.14.3 to 9.2.0 & also run tests with JDK 25 Nov 11, 2025
@hankem hankem requested a review from codecholeric November 11, 2025 07:26
@stefan-heilig-mw
Copy link

stefan-heilig-mw commented Nov 26, 2025

Hey @hankem & @codecholeric,

Thank you very much for this adaptation. Can you estimate when this PR will be reviewed?

@hankem
Copy link
Member Author

hankem commented Jan 27, 2026

I'm sorry @stefan-heilig-mw; I missed your message! 🙈

Just FYI: I think that the changes only affect the build internally; ArchUnit 1.4.1 already supports JDK 25 (#1440).

@stefan-heilig-mw
Copy link

Thank you for your reply. We have managed to get it working since then. There was an error in our setup, which initially pointed to ArchUnit.

as JDK 25 will add a call to `java.util.Objects.requireNonNull`
in the constructor of inner classes,
cf. https://inside.java/2025/04/04/quality-heads-up/
and https://bugs.openjdk.org/browse/JDK-8351274 .

Signed-off-by: Manfred Hanke <manfred.hanke@tngtech.com>
Signed-off-by: Manfred Hanke <manfred.hanke@tngtech.com>
Signed-off-by: Manfred Hanke <manfred.hanke@tngtech.com>
@hankem hankem changed the title Update Gradle Wrapper from 8.14.3 to 9.2.0 & also run tests with JDK 25 Update Gradle Wrapper from 8.14.3 to 9.3.1 & also run tests with JDK 25 Feb 11, 2026
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.

2 participants