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

chore: remove org.reflections:reflections #654

Merged
merged 3 commits into from
Nov 30, 2021
Merged

chore: remove org.reflections:reflections #654

merged 3 commits into from
Nov 30, 2021

Conversation

algomaster99
Copy link
Member

@algomaster99 algomaster99 commented Nov 29, 2021

-------------------------------------------------------
 D E P C L E A N   A N A L Y S I S   R E S U L T S
-------------------------------------------------------
USED DIRECT DEPENDENCIES [7]: 
        org.eclipse.jgit:org.eclipse.jgit:5.7.0.202003110725-r:compile (2 MB)
        org.sonarsource.java:java-checks:6.9.0.23563:compile (2 MB)
        fr.inria.gforge.spoon:spoon-core:10.0.1-SNAPSHOT:compile (1 MB)
        org.sonarsource.java:java-frontend:6.9.0.23563:compile (909 KB)
        info.picocli:picocli:4.5.2:compile (381 KB)
        commons-io:commons-io:2.7:compile (269 KB)
        org.json:json:20190722:compile (63 KB)
USED INHERITED DEPENDENCIES [0]: 
USED TRANSITIVE DEPENDENCIES [31]: 
        org.bouncycastle:bcprov-jdk15on:1.64:compile (4 MB)
        org.sonarsource.sonarqube:sonar-plugin-api:7.9:compile (3 MB)
        com.google.guava:guava:26.0-jre:compile (2 MB)
        com.fasterxml.jackson.core:jackson-databind:2.13.0:compile (1 MB)
        xerces:xercesImpl:2.12.0:compile (1 MB)
        org.apache.commons:commons-compress:1.21:compile (994 KB)
        org.apache.commons:commons-lang3:3.12.0:compile (573 KB)
        com.fasterxml.woodstox:woodstox-core:5.2.0:compile (506 KB)
        com.fasterxml.jackson.core:jackson-core:2.13.0:compile (365 KB)
        org.bouncycastle:bcpg-jdk15on:1.64:compile (321 KB)
        commons-lang:commons-lang:2.6:compile (277 KB)
        org.codehaus.plexus:plexus-utils:3.3.0:compile (257 KB)
        com.google.code.gson:gson:2.8.6:compile (234 KB)
        org.apache.maven:maven-model:3.8.4:compile (210 KB)
        org.sonarsource.sslr:sslr-core:1.23:compile (193 KB)
        org.checkerframework:checker-qual:2.5.2:compile (188 KB)
        org.codehaus.woodstox:stax2-api:4.1:compile (165 KB)
        com.googlecode.javaewah:JavaEWAH:1.1.7:compile (162 KB)
        org.apache.maven.shared:maven-shared-utils:3.3.3:compile (150 KB)
        com.fasterxml.jackson.core:jackson-annotations:2.13.0:compile (73 KB)
        org.sonarsource.analyzer-commons:sonar-analyzer-commons:1.12.0.632:compile (72 KB)
        com.martiansoftware:jsap:2.1:compile (67 KB)
        org.slf4j:slf4j-api:1.7.25:compile (40 KB)
        com.google.code.findbugs:jsr305:1.3.9:compile (32 KB)
        org.apache.maven.shared:maven-invoker:3.1.0:compile (31 KB)
        org.sonarsource.analyzer-commons:sonar-xml-parsing:1.12.0.632:compile (26 KB)
        com.google.errorprone:error_prone_annotations:2.1.3:compile (13 KB)
        org.sonarsource.analyzer-commons:sonar-analyzer-recognizers:1.12.0.632:compile (9 KB)
        com.google.j2objc:j2objc-annotations:1.1:compile (8 KB)
        javax.inject:javax.inject:1:compile (2 KB)
        org.sonarsource.java:jdt:shaded:6.9.0.23563 (size unknown)
POTENTIALLY UNUSED DIRECT DEPENDENCIES [2]: 
        org.reflections:reflections:0.9.12:compile (103 KB)
        org.sonarsource.java:java-checks-testkit:6.9.0.23563:compile (5 KB)
POTENTIALLY UNUSED INHERITED DEPENDENCIES [0]: 
POTENTIALLY UNUSED TRANSITIVE DEPENDENCIES [16]: 
        org.eclipse.jdt:org.eclipse.jdt.core:3.23.0:compile (6 MB)
        org.eclipse.platform:org.eclipse.osgi:3.17.0:compile (1 MB)
        org.eclipse.platform:org.eclipse.core.resources:3.15.100:compile (879 KB)
        org.bouncycastle:bcpkix-jdk15on:1.64:compile (857 KB)
        org.javassist:javassist:3.26.0-GA:compile (764 KB)
        cglib:cglib-nodep:3.2.5:compile (344 KB)
        org.eclipse.platform:org.eclipse.text:3.12.0:compile (288 KB)
        com.jcraft:jsch:0.1.55:compile (275 KB)
        org.eclipse.platform:org.eclipse.equinox.common:3.15.0:compile (139 KB)
        org.eclipse.platform:org.eclipse.equinox.preferences:3.9.0:compile (136 KB)
        org.eclipse.platform:org.eclipse.core.commands:3.10.100:compile (114 KB)
        org.eclipse.platform:org.eclipse.core.jobs:3.12.0:compile (108 KB)
        org.eclipse.platform:org.eclipse.core.contenttype:3.8.0:compile (100 KB)
        org.eclipse.platform:org.eclipse.core.runtime:3.23.0:compile (70 KB)
        com.jcraft:jzlib:1.1.1:compile (67 KB)
        org.codehaus.mojo:animal-sniffer-annotations:1.14:compile (3 KB)

I ran depclean v2.0.1 over sorald 8762596 and it generated the above report. This PR removes the Unused direct dependency.

Thanks for the amazing tool, @cesarsotovalero and all its contributors!

@algomaster99
Copy link
Member Author

algomaster99 commented Nov 29, 2021

I think depclean is falsely reporting org.sonarsource.java:java-checks-testkit:6.9.0.23563:compile (5 KB) as unused direct dependency. We use that package extensively here.

I accidentally force-pushed. The latest commit just adds again the removed dependency.

@algomaster99 algomaster99 marked this pull request as draft November 29, 2021 14:35
@algomaster99 algomaster99 changed the title chore: remove unused direct dependencies chore: remove org.reflections:reflections Nov 30, 2021
@algomaster99 algomaster99 marked this pull request as ready for review November 30, 2021 08:55
@algomaster99
Copy link
Member Author

@khaes-kth we can merge this PR. This package is not used. The tool and the tests confirm it.

@khaes-kth
Copy link
Collaborator

Great. Let's do it then!

Copy link
Collaborator

@khaes-kth khaes-kth left a comment

Choose a reason for hiding this comment

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

LGTM!

@khaes-kth khaes-kth merged commit 7344d92 into master Nov 30, 2021
@khaes-kth khaes-kth deleted the depclean branch November 30, 2021 09:06
@algomaster99
Copy link
Member Author

@khaes-kth deployment failed. I am looking into it.

@algomaster99
Copy link
Member Author

@khaes-kth I am can't be sure that the deployment required the presence of reflections package. Moreover, maven-gpg-plugin does not use this package too. Let's try to deploy it locally from your computer because I would need admin access for looking at all the keys required.

@khaes-kth
Copy link
Collaborator

@khaes-kth I am can't be sure that the deployment required the presence of reflections package. Moreover, maven-gpg-plugin does not use this package too. Let's try to deploy it locally from your computer because I would need admin access for looking at all the keys required.

Sure. We'll do it after our meeting today.

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

Successfully merging this pull request may close these issues.

2 participants