Skip to content

Conversation

@mernst
Copy link
Member

@mernst mernst commented Jan 17, 2026

No description provided.

@mernst mernst requested a review from smillst January 17, 2026 00:41
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 17, 2026

📝 Walkthrough

Walkthrough

Updates README text for jtreg usage (rewords bullets to include passing options, verifying duplicate errors, and running two separate compilation commands). Adds a new test for issue 7360 with three Java files: public class C (with public static C create() returning null), public class W (with public static W wrap(C) returning null), and package-private MyTest (method from() calling W.wrap(C.create())); test metadata specifies two compile steps using OptionalChecker. Modifies AnnotatedTypeMirror.hasExplicitAnnotation(...) to guard against a null primary annotation before checking explicit annotations.

Suggested reviewers

  • smillst
  • msridhar
🚥 Pre-merge checks | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

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.

3 participants