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

APOC to jdk 21 #688

Merged
merged 4 commits into from
Nov 19, 2024
Merged

APOC to jdk 21 #688

merged 4 commits into from
Nov 19, 2024

Conversation

gem-neo4j
Copy link
Contributor

@gem-neo4j gem-neo4j commented Nov 15, 2024

Neo has moved to Java 21, so we will follow suit!

Gradlew 8.5 is needed to get Java 21 full support.
There were some bugs in spotless so need to specify more specifically which versions to use.

Also need to version bump, 2024.12.0

@gem-neo4j
Copy link
Contributor Author

Move to Java 21?

@Lojjs Lojjs self-assigned this Nov 18, 2024
Copy link
Contributor

@Lojjs Lojjs left a comment

Choose a reason for hiding this comment

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

Except the place you missed to update, I think it looks good.

sourceCompatibility = JavaVersion.VERSION_17
targetCompatibility = JavaVersion.VERSION_17
sourceCompatibility = JavaVersion.VERSION_21
targetCompatibility = JavaVersion.VERSION_21
Copy link
Contributor

Choose a reason for hiding this comment

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

I found one additional place which mentions Java 17, the annotation in ApocProcessor.java

@gem-neo4j gem-neo4j merged commit ed7e897 into dev Nov 19, 2024
11 checks passed
@gem-neo4j gem-neo4j deleted the dev_bump_to_java_21 branch November 19, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants