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

Remove unused dependencies #135

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Celebrate-future
Copy link

@Celebrate-future Celebrate-future commented Feb 5, 2021

@disa6302 Hi, I am a user of project com.amazonaws:amazon-kinesis-video-streams-producer-sdk-java:1.10.0. I found that its pom file introduced 47 dependencies. However, among them, 9 libraries (19%) have not been used by your project (the redundant dependencies are listed below). This PR helps com.amazonaws:amazon-kinesis-video-streams-producer-sdk-java:1.10.0 lose weight :) I have tested the revised configuration in my local environment. It is safe to remove the unused libraries.

Best regards

Redundant dependencies----


com.google.code.findbugs:jsr305:jar:3.0.2:compile
com.google.errorprone:error_prone_annotations:jar:2.2.0:compile
com.google.guava:listenablefuture:jar:9999.0-empty-to-avoid-conflict-with-guava:compile
com.google.j2objc:j2objc-annotations:jar:1.1:compile
javax.measure:jsr-275:jar:0.9.1:compile
org.codehaus.mojo:animal-sniffer-annotations:jar:1.17:compile
com.amazonaws:jmespath-java:jar:1.11.602:compile
javax.inject:javax.inject:jar:1:compile
com.google.inject.extensions:guice-multibindings:jar:4.2.3:compile

Outdated dependencies

org.codehaus.mojo:animal-sniffer-annotations:1.17 (1863 days without maintenance)
javax.inject:javax.inject:1 (5036 days without maintenance)
com.google.j2objc:j2objc-annotations:1.1 (2558 days without maintenance)
com.google.inject.extensions:guice-multibindings:4.2.3 (1226 days without maintenance)
com.amazonaws:jmespath-java:1.11.602 (1459 days without maintenance)

@disa6302
Copy link
Contributor

disa6302 commented Feb 5, 2021

Thank you @Celebrate-future . I will do some testing on my end to confirm the changes and get back ASAP.

@Celebrate-future
Copy link
Author

@disa6302 Thanks for your feedback :)

pom.xml Outdated
<groupId>com.amazonaws</groupId>
<artifactId>amazon-kinesis-video-streams-producer-sdk-java</artifactId>
<name>Amazon Kinesis Video Streams Producer SDK Java</name>
<version>1.10.0</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

The current version is 1.11.0. Can you please change the version back to that?

Copy link
Author

Choose a reason for hiding this comment

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

OK. I will change it back.

@disa6302
Copy link
Contributor

@Celebrate-future ,

The changes work. I tested it out. One comment : We are on version 1.11.0. Please change it from 1.10.0 to 1.11.0 in the PR and we are all set to merge this.

@Celebrate-future
Copy link
Author

I have already changed the version back to 1.11.0.
Thanks.

@disa6302 disa6302 changed the base branch from master to develop December 6, 2023 20:34
@disa6302 disa6302 changed the base branch from develop to master December 6, 2023 20:36
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