Skip to content

Conversation

@marcindabrowski
Copy link
Contributor

In v2.37.0 the annotations that were previously in type_annotations artifact have been merged into annotations. Annotations added there:

  • com.google.errorprone.annotations.ImmutableTypeParameter,
  • com.google.errorprone.annotations.ThreadSafeTypeParameter.

In next releases only Javadoc was changed.

This update will enable updates of gson and guava libraries.

In v2.37.0 the annotations that were previously in type_annotations artifact have been merged into annotations.
Annotations added there:
* com.google.errorprone.annotations.ImmutableTypeParameter,
* com.google.errorprone.annotations.ThreadSafeTypeParameter.

In next releases only Javadoc was changed.

This update is enable updates of gson and guava libraries
Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

gRPC wasn't using the type parameters, gRPC not having upgraded yet shouldn't impact anyone; everyone is free to upgrade error prone annotations themselves.

This change looks fine, although I'm upgrading all dependencies in #12522 as part of our post-release process, so we'll probably just favor that wider change instead of this one. There were the comments talking about how newer versions of error prone annotations caused trouble, but that seems to no longer be a problem, and so I've removed those comments/restrictions in my upgrade.

@marcindabrowski
Copy link
Contributor Author

marcindabrowski commented Nov 19, 2025

Great to see so many bumps.
@ejona86 Can you share why not upgrade Netty to 4.1.128.Final?

@marcindabrowski marcindabrowski deleted the deps/bump-errorprone-annotations_2.44.0 branch November 19, 2025 19:46
@ejona86
Copy link
Member

ejona86 commented Nov 19, 2025

We always bump Netty separately. It undergoes a lot more testing.

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