Skip to content

Test on JDK 27 EA#1540

Merged
msridhar merged 7 commits intomasterfrom
test-on-jdk-27
Apr 20, 2026
Merged

Test on JDK 27 EA#1540
msridhar merged 7 commits intomasterfrom
test-on-jdk-27

Conversation

@msridhar
Copy link
Copy Markdown
Collaborator

@msridhar msridhar commented Apr 19, 2026

Run our tests on the latest JDK 27 EA build.

Also fixes our check for explicit @Nullable annotations on lambda parameters, handling a change in the types javac provides in JDK 27.

Summary by CodeRabbit

  • Tests

    • Extended CI/CD testing to include JDK 27 Early Access in addition to existing versions, ensuring compatibility across a wider range of Java environments.
  • Refactor

    • Improved detection of explicit nullability annotations on lambda parameters for more accurate analysis with newer JDK releases.

@msridhar msridhar marked this pull request as draft April 19, 2026 22:15
@msridhar msridhar changed the title Test on JDK 27 EA [draft] Test on JDK 27 EA Apr 19, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.48%. Comparing base (15a35d1) to head (4069828).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1540   +/-   ##
=========================================
  Coverage     88.48%   88.48%           
  Complexity     2843     2843           
=========================================
  Files           103      103           
  Lines          9489     9490    +1     
  Branches       1905     1905           
=========================================
+ Hits           8396     8397    +1     
  Misses          531      531           
  Partials        562      562           

☔ View full report in Codecov by Sentry.
📢 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.

@msridhar msridhar changed the title [draft] Test on JDK 27 EA Test on JDK 27 EA Apr 20, 2026
@msridhar msridhar marked this pull request as ready for review April 20, 2026 16:26
Base automatically changed from wala-1.7.1 to master April 20, 2026 17:14
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: fa7fbe39-cdb1-4d10-aeb2-2b97b52c03c9

📥 Commits

Reviewing files that changed from the base of the PR and between 15a35d1 and 4069828.

📒 Files selected for processing (4)
  • .github/workflows/continuous-integration.yml
  • buildSrc/src/main/groovy/nullaway.java-test-conventions.gradle
  • jar-infer/jar-infer-lib/build.gradle
  • nullaway/src/main/java/com/uber/nullaway/dataflow/CoreNullnessStoreInitializer.java

Walkthrough

This PR adds JDK 27 EA support across the build infrastructure and adjusts lambda-parameter nullability detection. Changes include: (1) updating the CI workflow to explicitly install JDK 27 EA before the default JDK setup, (2) extending the Gradle test-conventions matrix to generate a testJdk27 test task, (3) skipping testJdk27 in the jar-infer build, and (4) narrowing lambda-parameter explicit-nullability detection in CoreNullnessStoreInitializer to only check annotations directly present on the symbol.

Possibly related PRs

Suggested reviewers

  • yuxincs
  • lazaroclapp
  • seemantasaha
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Test on JDK 27 EA' accurately summarizes the primary objective of the PR, which is to add testing support for JDK 27 Early Access.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test-on-jdk-27

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@msridhar msridhar enabled auto-merge (squash) April 20, 2026 17:32
@msridhar msridhar merged commit e3e78aa into master Apr 20, 2026
12 checks passed
@msridhar msridhar deleted the test-on-jdk-27 branch April 20, 2026 17:32
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