ci: revert JDK 25 bump, restore pinned JDK 17 - #2044
Conversation
Reverts #2038. We intentionally pin JDK 17 in these workflows for API compatibility testing. Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe merge and release GitHub Actions workflows now provision Temurin JDK 17 instead of JDK 25. All other workflow steps remain unchanged. ChangesJDK Version Update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to This change restores the intended JDK 17 CI and release environment without an identified current-head risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2044 +/- ##
============================================
+ Coverage 92.70% 93.27% +0.57%
- Complexity 730 732 +2
============================================
Files 60 60
Lines 1741 1741
Branches 203 203
============================================
+ Hits 1614 1624 +10
+ Misses 77 69 -8
+ Partials 50 48 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|



Reverts #2038, which was auto-created and automerged by renovate's new
uses-withruntime-version detection and broke the build. We pin JDK 17 in these workflows intentionally for API compatibility testing.Org-level prevention: open-feature/community-tooling#127 disables
uses-withdeps so renovate no longer touches runtime versions in action inputs.