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 28 forks source link

Do not log empty line when unused warnings are not logged #296

Closed remcomokveld closed 7 months ago

remcomokveld commented 7 months ago

With the changes in https://github.com/cashapp/licensee/pull/285 there's now scenarios where only the empty-line is logged at lifecycle level. When using a rich console this results in the task appearing in the output even though it doesn't really log anything.

I am not completely happy with the way that I adjusted the test because it could be that there's another reason for an extra empty line but also wan't sure how this could be tested better. At least the test failed before making the change and after the change is passes