Skip to content

Could com.wdbyte.core-java-modules:core-java-io:1.0.0-SNAPSHOT drop off redundant dependencies? #1

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

Merged
merged 1 commit into from
Apr 20, 2022

Conversation

Celebrate-future
Copy link
Contributor

@Celebrate-future Celebrate-future commented Apr 18, 2022

image
This figure presents the dependency tree between multiple modules in core-java-modules. As shown in this figure, Libraries

com.google.code.findbugs:jsr305:jar:3.0.2:compile
com.google.guava:guava:jar:31.0.1-jre:compile
com.google.guava:failureaccess:jar:1.0.1:compile
com.google.errorprone:error_prone_annotations:jar:2.7.1:compile
com.google.guava:listenablefuture:jar:9999.0-empty-to-avoid-conflict-with-guava:compile
org.checkerframework:checker-qual:jar:3.12.0:compile
com.google.j2objc:j2objc-annotations:jar:1.3:compile

Outdated dependencies

com.google.j2objc:j2objc-annotations:1.3 (2382 days without maintenance)
com.google.guava:failureaccess:1.0.1 (1712 days without maintenance)


in module core-java-performance, core-java-8, core-java-9, core-java-performance-code, core-java-io, core-java-string is inherited from their parent module. However, it is not used by module core-java-io. We can perform refactoring operations in the pom, by removing such redundant dependencies in module core-java-io.

Specifically, the scope of com.google.guava:guava in core-java-io can be changed from compile to provided. The revisions in the pom are described as follows:
image

@Celebrate-future Celebrate-future changed the title remove redundant dependencies Could com.wdbyte.core-java-modules:core-java-io:1.0.0-SNAPSHOT drop off redundant dependencies? Apr 18, 2022
@niumoo niumoo merged commit fe89918 into niumoo:master Apr 20, 2022
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