Skip to content

Upgrade minimum JDK to 17 and refresh dependencies#830

Open
HTHou wants to merge 2 commits into
developfrom
codex/upgrade-jdk17-logback-antlr
Open

Upgrade minimum JDK to 17 and refresh dependencies#830
HTHou wants to merge 2 commits into
developfrom
codex/upgrade-jdk17-logback-antlr

Conversation

@HTHou
Copy link
Copy Markdown
Contributor

@HTHou HTHou commented Jun 5, 2026

Summary

  • raise the Java build target and minimum runtime checks to JDK 17
  • update Logback to 1.5.32, ANTLR 4 to 4.13.2, and align SLF4J to 2.0.17
  • update CI workflows and user-facing docs to use JDK 17 as the minimum
  • address open Maven Dependabot alerts by upgrading Hadoop, Jackson, Commons Compress, Guava, and convergence pins

Validation

  • ./mvnw -P with-java -DskipTests package
  • git diff --check

Note: OSS Index returned HTTP 401 warnings during the Maven run, but the enforcer rule completed without failing the build.

@HTHou HTHou marked this pull request as ready for review June 5, 2026 10:32
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.58%. Comparing base (2a864c5) to head (7bda5ee).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #830   +/-   ##
========================================
  Coverage    61.57%   61.58%           
========================================
  Files          731      731           
  Lines        45874    45871    -3     
  Branches      6880     6880           
========================================
  Hits         28249    28249           
+ Misses       16614    16611    -3     
  Partials      1011     1011           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants