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

Report unused exclude clauses in dependencies #1368

Open
boris-petrov opened this issue Feb 2, 2025 · 3 comments
Open

Report unused exclude clauses in dependencies #1368

boris-petrov opened this issue Feb 2, 2025 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@boris-petrov
Copy link

Is your feature request related to a problem? Please describe.
It's not a problem but is a nice feature to have.

Describe the solution you'd like
Say I have this script:

implementation('some:dependency:1.0.0') {
  exclude group: 'this.is.not.actually.a', module: 'dependency'
}

some:dependency doesn't have this.is.not.actually.a:dependency as a dependency. Or it used to but I've updated it to a new version and now it doesn't. It would be nice to get a report that this exclusion is unneeded.

Describe alternatives you've considered
N/A

Additional context
Gradle Lint has a similar feature.

@autonomousapps autonomousapps added the enhancement New feature or request label Feb 9, 2025
@autonomousapps autonomousapps added this to the unplanned milestone Feb 9, 2025
@autonomousapps
Copy link
Owner

Thanks for the request! It is unlikely to be worked on in the foreseeable future. It might also be out of scope, but I'm not totally sure of that and could be convinced it's a good feature for this plugin. My intuition says it belongs in something separate (like Gradle Lint). In fact, is there a reason Gradle Lint isn't working for you?

@boris-petrov
Copy link
Author

Thanks for the response! Well, Gradle Lint has so many issues I don't know where to begin. For example this one. I don't think they'll ever fix them.

As for why that's a good feature - well, dependency-analysis-gradle-plugin is a Gradle plugin for analyzing dependencies, right? It's in the name! And unused exclude clauses are a problem that concern dependencies. So it's kind of related. :) It's nothing urgent but I think it would be a nice addition to the already big list of nice features of this plugin.

@autonomousapps
Copy link
Owner

Thanks for the response! I won't have time for this, but I could be open to a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants