-
Notifications
You must be signed in to change notification settings - Fork 486
Closed
Labels
Description
Some of our formatters can't compile on Java 11 anymore, so supporting Java 11 at all is getting harder. We'll support it a while longer, but its days are numbered...
- would make this issue trivial: Remove
toLines(
and usetoContent("""
instead #1533
Running checklist of todo items to make this happen (please don't send PRs for these at this time, we aren't pulling the plug on Java 11 yet)
- remove CDT build complexity added in aab831cbump to latest JGit, which has worktree support
can remove JGit system config workaround in 0c66cb0nope still need itwe'll need to bump minimum Gradle to 7.3, which lets us remove some compat stuffTo pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.update documented requirements for gradle and mavenTo pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
jochenberger, coderkun, simschla, alcarraz and slachiewicz
Metadata
Metadata
Assignees
Labels
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
Remove the Eclipse CDT stuff from the build and tests on Java 11. We …
nedtwigg commentedon Jan 6, 2025
Not sure why, but on Java 11, JGit was causing this to happen.
We don't have the problem on Java 17 or later. This hack fixes it
0c66cb0
But it would probably be good to remove it when we bump the minimum Java from 11 to 17.
EDIT: after removing it, we get the error Java 17 only, but not 21 and 24. Hrm... I wonder if it is some kind of global build cache thing - first build to run hits the problem, subsequent works around thanks to reusing a result from the remote build cache...
nedtwigg commentedon Jan 7, 2025
Another benefit to bumping to Java 17 is that we could use the latest JGit, which adds worktree support.
nedtwigg commentedon Jan 7, 2025
It would also bump our required version of Gradle to 7.3, which would allow some minor simplifications here:
spotless/plugin-gradle/src/main/java/com/diffplug/gradle/spotless/SpotlessPlugin.java
Lines 30 to 31 in f22eda9
spotless/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/GradleIntegrationHarness.java
Lines 46 to 50 in f22eda9
nedtwigg commentedon Jul 16, 2025
With the pending release of Gradle 9, I think this is the right time for us to bump minimums, thanks to the linked PR by @Goooler above. If you need/want us to stay compatible with old stuff for a bit longer, speak up now!
Goooler commentedon Jul 17, 2025
I don't.
nedtwigg commentedon Jul 18, 2025
My plan is:
Bump the min Gradle and Java requirements to 7.3 and 17 (#2540, first…
nedtwigg commentedon Jul 21, 2025
Complete! Huge thanks @Goooler, you did the hard part!