Skip to content

Use Error Prone 2.46.0#7438

Merged
mernst merged 2 commits intotypetools:masterfrom
mernst:errorprone-2460-2
Jan 9, 2026
Merged

Use Error Prone 2.46.0#7438
mernst merged 2 commits intotypetools:masterfrom
mernst:errorprone-2460-2

Conversation

@mernst
Copy link
Member

@mernst mernst commented Jan 9, 2026

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 9, 2026

📝 Walkthrough

Walkthrough

This pull request updates error-prone related dependencies and improves documentation formatting. The net-ltgt-errorprone plugin is bumped from version 4.3.0 to 4.4.0, and error-prone-annotations is updated from 2.45.0 to 2.46.0 in the gradle/libs.versions.toml file. Additionally, a comment in build.gradle is modified to capitalize "Error Prone" and adjust the wording for consistency. These changes do not alter any method signatures, public APIs, or execution logic.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
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.


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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @gradle/libs.versions.toml:
- Line 7: The net-ltgt-errorprone plugin version is incorrect (uses "4.4.0"
which doesn't exist); update the dependency entry named net-ltgt-errorprone in
gradle/libs.versions.toml to use version "4.3.0" instead so the declared version
matches the latest available release.
📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aaf33ab and db78dcd.

📒 Files selected for processing (2)
  • build.gradle
  • gradle/libs.versions.toml
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: mernst
Repo: typetools/checker-framework PR: 7354
File: annotation-file-utilities/tests/LocalMultipleManyMethodsShifted.java:14-14
Timestamp: 2025-11-02T02:18:00.536Z
Learning: In the Checker Framework repository, when doing refactoring PRs (such as preferring isEmpty() over size() comparisons), test files should not be changed. Tests should remain stable to preserve their exact patterns for annotation processing and bytecode verification purposes.
🔇 Additional comments (1)
build.gradle (1)

205-207: LGTM!

The comment capitalization has been corrected to use the proper product name "Error Prone" for consistency.

@shubham5080

This comment was marked as off-topic.

@mernst
Copy link
Member Author

mernst commented Jan 9, 2026

@shubham5080 If that is the way that you are asking for help -- spamming an unrelated issue and giving no details about what you need help with -- then you are not a good candidate for GSoC.

@mernst mernst merged commit cf9f354 into typetools:master Jan 9, 2026
21 checks passed
@mernst mernst deleted the errorprone-2460-2 branch January 9, 2026 18:33
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