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 Java 21 compatibilities #1920

Merged
merged 10 commits into from
Dec 3, 2023
Merged

Conversation

Goooler
Copy link
Member

@Goooler Goooler commented Dec 1, 2023

@Goooler

This comment was marked as outdated.

@Goooler Goooler changed the title Bump palantir-java-format for Java 21 Use palantir-java-format 2.38.0 on Java 21 Dec 1, 2023
Co-authored-by: Goooler <wangzongler@gmail.com>
@steven-sheehy
Copy link
Contributor

This doesn't seem to have the java 21 workflow or the guava variant fix?

@Goooler
Copy link
Member Author

Goooler commented Dec 2, 2023

Yeah, I should probably add Java 21 to CI first.

@Goooler Goooler marked this pull request as draft December 2, 2023 04:02
* Fix CombinedJavaFormatStepTest

* Fix GoogleJavaFormatStepTest

* Fix PalantirJavaFormatStepTest

* Fix GoogleJavaFormatIntegrationTest

* Fix MultiProjectTest

* Fix MavenProvisionerTest

* Fix PalantirJavaFormatTest

* Fix SpecificFilesTest

* Fix GoogleJavaFormatTest
@Goooler Goooler changed the title Use palantir-java-format 2.38.0 on Java 21 Fix Java 21 compatibilities Dec 3, 2023
@Goooler Goooler marked this pull request as ready for review December 3, 2023 05:46
@Goooler
Copy link
Member Author

Goooler commented Dec 3, 2023

Fixed all tests but jvmLocalCache, which is related to #987, no idea for this.

https://scans.gradle.com/s/6mslfqjw7nhie/tests/overview?outcome=FAILED

@nedtwigg
Copy link
Member

nedtwigg commented Dec 3, 2023

Fascinating. The JvmLocalCache test fails only on Java 21, and I can replicate it on my machine. I'm digging in...

…est JvmLocalCache doesn't work with Gradle 8.5.
@nedtwigg nedtwigg mentioned this pull request Dec 3, 2023
1 task
@nedtwigg nedtwigg merged commit 7717dba into diffplug:main Dec 3, 2023
13 checks passed
@Goooler Goooler deleted the bump-palantir-java-format branch December 4, 2023 00:36
@nedtwigg
Copy link
Member

nedtwigg commented Dec 4, 2023

Published in plugin-gradle 6.23.3 and plugin-maven 2.41.1.

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.

Test integration with the latest Java version on CI Update default versions for palantir-java-format
4 participants