-
Notifications
You must be signed in to change notification settings - Fork 75
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
base: master
Are you sure you want to change the base?
Conversation
Thank you @Celebrate-future . I will do some testing on my end to confirm the changes and get back ASAP. |
@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> |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
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. |
I have already changed the version back to 1.11.0. |
@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----
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)