cashapp / licensee

Gradle plugin which validates the licenses of your dependency graph match what you expect
https://cashapp.github.io/licensee/docs/1.x/
Apache License 2.0
637 stars 30 forks source link

Add ability to set Configuration for licensee #75

Open lhwdev opened 2 years ago

lhwdev commented 2 years ago

I have one copy-pasted dependency which is not (should not) included in configuration, but I need it to included in information so that it is validated and included into artifacts.json. I think it would be great if I:

JakeWharton commented 2 years ago

Hmm the proposed DSL doesn't really scale to the different usages of the plugin. In projects with Android, Kotlin multiplatform, or the combination of both there is a matrix of configurations which are used and sourced from the plugins themselves.

I'll have to think about this case.

lhwdev commented 2 years ago

I see. So, how about combining the extra configurations, like

So if you use KMP + Android, you get all three configurations above combine. Eventually just adding two List<Configuration>. I think replacing original configuration is not needed(we have ignoreDependency), so we only need to do this for extra configurations.

hfhbd commented 1 year ago

This is now possible by creating your own licensee tasks by using the configrationToCheck function, see #191.