Fix release build: upgrade maven-compiler-plugin and consolidate Java compiler release target#802
Open
jordanpadams wants to merge 2 commits into
Open
Fix release build: upgrade maven-compiler-plugin and consolidate Java compiler release target#802jordanpadams wants to merge 2 commits into
jordanpadams wants to merge 2 commits into
Conversation
Upgrade maven-compiler-plugin from 3.8.1 to 3.13.0 in root pom to support Java 21 release target. Remove per-submodule maven.compiler.release overrides so all modules inherit the single source-of-truth (release 17) from the root pom, eliminating the divergence that broke the stable release build. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



🗒️ Summary
Fixes a release build failure caused by
maven-compiler-plugin:3.8.1not supporting Java release target 21. Theservicemodule overrodemaven.compiler.releaseto21, but the old plugin version predates Java 21 support, producing:Fatal error compiling: error: release version 21 not supported.Changes:
maven-compiler-pluginfrom3.8.1→3.13.0in rootpom.xml(adds Java 21 support)maven.compiler.release=21override fromservice/pom.xml— no Java 21 language features are in use; service runs on JVM 21 at runtime viaBP_JVM_VERSIONmaven.compiler.release=17frommodel/pom.xml— already the root defaultmaven-compiler-plugin.versionproperty fromlexer/pom.xmlAll submodules now inherit
maven.compiler.release=17from the root pom as a single source of truth.🤖 AI Assistance Disclosure
Estimated % of code influenced by AI: 75%
⚙️ Test Data and/or Report
Build configuration fix only — no application logic changed. Validation is that the stable release build which previously failed with
release version 21 not supportedshould now succeed.Note: Integration tests are failing, but not sure there is much we can do about that right now with the repos a bit out of sync with backwards-incompatible changes
♻️ Related Issues
N/A — fixing stable release pipeline regression.
🤓 Reviewer Checklist
Reviewers: Please verify the following before approving this pull request.
Documentation and PR Content
Security & Quality
Testing & Validation
Maintenance