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

Adds type_annotations back but as a relocation to annotations #4906

Closed
wants to merge 1 commit into from

Conversation

tbroyer
Copy link
Contributor

@tbroyer tbroyer commented Mar 17, 2025

Adds a dependency to type_annotations into annotations to make sure it'll "update" an older type_annotations dependency present in the same "bag" of dependencies (works best in Gradle; Maven having a "nearest definition" rule so won't always work but hopefully people align versions one way or another).

Adds a dependency to type_annotations into annotations to make
sure it'll "update" an older type_annotations dependency present
in the same "bag" of dependencies (works best in Gradle; Maven
having a "nearest definition" rule so won't always work but
hopefully people align versions one way or another).
@tbroyer
Copy link
Contributor Author

tbroyer commented Mar 17, 2025

With Gradle, the relocation (being always present in the resolution) will print a note when run with --info logs (or more verbose).

With Maven, it'll only generate a warning if the type_annotations is directly present in the pom.xml (from my tests)

copybara-service bot pushed a commit that referenced this pull request Mar 17, 2025
Adds a dependency to type_annotations into annotations to make sure it'll "update" an older type_annotations dependency present in the same "bag" of dependencies (works best in Gradle; Maven having a "nearest definition" rule so won't always work but hopefully people align versions one way or another).

Fixes #4906

FUTURE_COPYBARA_INTEGRATE_REVIEW=#4906 from tbroyer:type_annotations-relocation 31c2f10
PiperOrigin-RevId: 737627376
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