-
Notifications
You must be signed in to change notification settings - Fork 1k
fix(deps): update errorproneversion to v2.46.0 #15799
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Breaking changes from a minor release of errorprone.....what's this world coming to. 🤯 |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
breedx-splk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I did the right thing, but keep me honest. 🙏🏻
| tasks { | ||
| withType<JavaCompile>().configureEach { | ||
| with(options) { | ||
| compilerArgs.add("-XDaddTypeAnnotationsToSymbol=true") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it'd be interesting to see how much bytecode bloat this introduces. Not something blocking; just for educational purposes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah I wouldn't expect it to be too much at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reading https://bugs.openjdk.org/browse/JDK-8225377 it looks to me like it won't affect the generated bytecode.
This PR contains the following updates:
2.45.0→2.46.02.45.0→2.46.02.45.0→2.46.0Release Notes
google/error-prone (com.google.errorprone:error_prone_test_helpers)
v2.46.0: Error Prone 2.46.0Changes:
-XDaddTypeAnnotationsToSymbol=trueis now required for Error Prone invocations on JDK 21, to enable the javac fix for JDK-8225377: type annotations are not visible to javac plugins across compilation boundaries. See #5426 for details.valueattribute from@IncompatibleModifiersand@RequiredModifiers(#2122)EndPosTable,DiagnosticPosition) (5440bb4,06c2905,f3915ec)New checks:
DuplicateAssertion: detect duplicated assertion lines where the argument toassertThatis pureIfChainToSwitch: suggest converting chains of if-statements into arrow switchesScannerUseDelimiter: discourageScanner.useDelimiter("\\A")AddNullMarkedToClass: refactoring to add@NullMarkedannotation to top level classesConfiguration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.