mojohaus / license-maven-plugin

Maven plugin to download and collect license files from project dependencies.
https://www.mojohaus.org/license-maven-plugin/
GNU Lesser General Public License v3.0
107 stars 126 forks source link

Don't use getLog().isDebugEnabled() to determine the "verbose" flag #340

Open digulla opened 5 years ago

digulla commented 5 years ago

The code currently calls getLog().isDebugEnabled() to set the verbose flag.

In the code, I see patterns like these:

This feels like a hack to get logging at a level between INFO and DEBUG.

I suggest to get rid of the "verbose" flag and always log at DEBUG level for these.

For the blocks that prepare a complex string, use log.isDebugEnabled() directly.

ppalaga commented 5 years ago

What would happen with the verbose Mojo parameter?

slawekjaranowski commented 1 year ago

What would happen with the verbose Mojo parameter?

verbose Mojo parameter can be deprecated.