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

Enabling compilation with JDK23+ #153

Merged
merged 2 commits into from
Nov 13, 2024
Merged

Conversation

Jonas-Isr
Copy link
Member

Context

AI/ai-sdk-java-backlog#ISSUENUMBER. (no issue number)

Adds compatibility between Lombok and JDK23.
(For more info see here.)

Feature scope:

(not applicable)

Definition of Done

(not applicable)

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Aligned changes with the JavaScript SDK
  • Documentation updated
  • Release notes updated

@Jonas-Isr Jonas-Isr added the please-review Request to review a pull-request label Nov 12, 2024
@@ -53,6 +53,7 @@
<maven.compiler.source>17</maven.compiler.source>
<maven.compiler.target>17</maven.compiler.target>
<maven.compiler.release>17</maven.compiler.release>
<maven.compiler.proc>full</maven.compiler.proc>
Copy link
Contributor

@newtork newtork Nov 12, 2024

Choose a reason for hiding this comment

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

Annotation processing:

  • none - no annotation processing is performed.
  • only - only annotation processing is done, no compilation.
  • full - annotation processing and compilation.

full is the default. Starting with JDK 21 (sic), this option must be set explicitly.

More details here:
https://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html#proc

Copy link
Contributor

@newtork newtork left a comment

Choose a reason for hiding this comment

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

Please fix PR title. Essentially you are enabling compilation with JDK23+.

@Jonas-Isr Jonas-Isr changed the title add maven property Enabling compilation with JDK23+ Nov 13, 2024
@Jonas-Isr Jonas-Isr merged commit d49b498 into main Nov 13, 2024
5 checks passed
@Jonas-Isr Jonas-Isr deleted the fix-lombok-jdk23-incompatibility branch November 13, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
please-review Request to review a pull-request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants