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.

Sign up for GitHub

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

Jump to bottom

Removed guava from swagger-jaxrs #3142

Closed

albertotn wants to merge 1 commit into swagger-api:1.5 from albertotn:1.5

albertotn commented Mar 1, 2019 •

edited

Loading

My proposal is to remove guava from swagger-jaxrs, because:

not needed at compile time
not needed at test time
not needed a runtime

As main benefit, guava jar will not be part of dependencies and also all transitive dependencies


          Removed guava

576e354

guava is not used by this module, so is not neeed

frantuma force-pushed the 1.5 branch from cb6fef8 to 576e354 Compare

January 26, 2024 06:36

Member

frantuma commented Jan 26, 2024

closing as guava is used by module (even if removed would be picked as transitive deps from core

frantuma closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet