Skip to content
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

Upgrade Error Prone 2.21.1 -> 2.22.0 #800

Merged
merged 3 commits into from
Oct 4, 2023

Conversation

Picnic-Bot
Copy link
Contributor

@Picnic-Bot Picnic-Bot commented Sep 22, 2023

This PR contains the following updates:

Package Type Update Change
com.google.errorprone:error_prone_annotations (source) compile minor 2.21.1 -> 2.22.0
com.google.errorprone:error_prone_core (source) minor 2.21.1 -> 2.22.0
com.google.errorprone:error_prone_test_helpers (source) compile minor 2.21.1 -> 2.22.0
com.google.errorprone:error_prone_core (source) compile minor 2.21.1 -> 2.22.0
com.google.errorprone:error_prone_check_api (source) compile minor 2.21.1 -> 2.22.0
com.google.errorprone:error_prone_annotation (source) compile minor 2.21.1 -> 2.22.0

Release Notes

google/error-prone (com.google.errorprone:error_prone_annotations)

v2.22.0: Error Prone 2.22.0

Compare Source

We are considering raising the minimum supported JDK from JDK 11 to JDK 17 in a future release of Error Prone, see #​3803. Note that using a newer JDK version to run javac during the build doesn't prevent building code that is deployed to earlier versions, for example it's supported to use the JDK 17 javac and pass --release 11 to compile Java 11 code that is deployed to a JDK 11 runtime. If you have feedback, please comment on #​3803.

New checks:

Bug fixes and improvements:

  • Don't complain about literal IP addresses in AddressSelection (google/error-prone@44b6552)
  • Prevent SuggestedFixes#renameMethod from modifying return type declaration (#​4043)
  • Fix UnusedVariable false positives for private record parameters (#​2713)
  • When running in conservative mode, no longer assume that implementations of Map.get, etc. return null (#​2910)
  • CanIgnoreReturnValueSuggester: Support additional exempting method annotations (#​4009)
  • UnusedVariable: exclude junit5's @RegisterExtension (#​3892)
  • Support running all available patch checks (#​947)
  • Upgrade java-diff-utils 4.0 -> 4.12 (#​4081)
  • Flag unused Refaster template parameters (#​4060)
  • Support @SuppressWarnings("all") (#​4065)
  • Prevent Refaster UMemberSelect from matching method parameters (#​2456)
  • MissingDefault : Don't require // fall out comments on expression switches (#​2709)
  • Skip UnnecessaryLambda findings for usages in enhanced for loops (#​2518)
  • Fix bug where nested MissingBraces violations' suggested fixes result in broken code (#​3797)
  • Add support for specifying exemptPrefixes/exemptNames for UnusedVariable via flags (#​2753)
  • UnusedMethod: Added exempting variable annotations (#​2881)

Full Changelog: google/error-prone@v2.21.1...v2.22.0


  • If you want to rebase/retry this PR, check this box

@Picnic-Bot
Copy link
Contributor Author

Picnic-Bot commented Sep 22, 2023

Suggested commit message:

Upgrade Error Prone 2.21.1 -> 2.22.0 (#800)

See:
- https://github.com/google/error-prone/releases/tag/v2.22.0
- https://github.com/google/error-prone/compare/v2.21.1...v2.22.0
- https://github.com/PicnicSupermarket/error-prone/compare/v2.21.1-picnic-1...v2.22.0-picnic-1

@github-actions
Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@github-actions
Copy link

Looks good. All 5 mutations in this change were killed.

class surviving killed
🎉tech.picnic.errorprone.bugpatterns.LexicographicalAnnotationAttributeListing 0 2
🎉tech.picnic.errorprone.bugpatterns.util.Flags 0 3

Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

1 similar comment
@github-actions
Copy link

Looks good. All 5 mutations in this change were killed.

class surviving killed
🎉tech.picnic.errorprone.bugpatterns.LexicographicalAnnotationAttributeListing 0 2
🎉tech.picnic.errorprone.bugpatterns.util.Flags 0 3

Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

Copy link
Member

@Stephan202 Stephan202 left a comment

Choose a reason for hiding this comment

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

Added some commits.

Comment on lines -222 to +226
flags.getList(INCLUDED_ANNOTATIONS_FLAG), excludedAnnotations(flags));
flags.get(INCLUDED_ANNOTATIONS_FLAG).isPresent()
? Optional.of(flags.getListOrEmpty(INCLUDED_ANNOTATIONS_FLAG))
: Optional.empty(),
excludedAnnotations(flags));
Copy link
Member

Choose a reason for hiding this comment

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

This is not great, but alas.

@Stephan202 Stephan202 added this to the 0.14.0 milestone Sep 22, 2023
@Picnic-Bot
Copy link
Contributor Author

Edited/Blocked Notification

Renovate 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.

Warning: custom changes will be lost.

@Stephan202 Stephan202 force-pushed the renovate/version.error-prone-orig branch from 463fc28 to e1399c6 Compare October 4, 2023 10:36
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Looks good. All 5 mutations in this change were killed.

class surviving killed
🎉tech.picnic.errorprone.bugpatterns.LexicographicalAnnotationAttributeListing 0 2
🎉tech.picnic.errorprone.bugpatterns.util.Flags 0 3

Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@rickie rickie force-pushed the renovate/version.error-prone-orig branch from e1399c6 to ac6cbbc Compare October 4, 2023 11:45
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Looks good. All 5 mutations in this change were killed.

class surviving killed
🎉tech.picnic.errorprone.bugpatterns.LexicographicalAnnotationAttributeListing 0 2
🎉tech.picnic.errorprone.bugpatterns.util.Flags 0 3

Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

Copy link
Member

@rickie rickie left a comment

Choose a reason for hiding this comment

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

Tweaked suggested commit message to be similar to previous commits.

@rickie rickie merged commit ef75b80 into master Oct 4, 2023
17 checks passed
@rickie rickie deleted the renovate/version.error-prone-orig branch October 4, 2023 12:43
@Stephan202 Stephan202 changed the title Upgrade version.error-prone-orig 2.21.1 -> 2.22.0 Upgrade Error Prone 2.21.1 -> 2.22.0 Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants