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
626 stars 29 forks source link

No license found for compose (desktop) modules #58

Closed warting closed 2 years ago

warting commented 2 years ago

I am using composereorderable (https://repo1.maven.org/maven2/org/burnoutcrew/composereorderable/reorderable/0.6.2/) and this tool says it has no licenses even though it has the apache 2 license.

The reason seems to be that one of the artifacts is not having any resolvedFiles. If it has no files, it is safe to assume that no license is needed for that? I've made a PR that solves my issue, but I'm not sure it is the correct approach to solve it.

Please have a look https://github.com/cashapp/licensee/pull/57

JakeWharton commented 2 years ago

Going to close as a duplicate of #30 which is fixed in 1.5.0. Please try and let me know if it resolves this. Otherwise we'll have to get a failing test case integrated.