-
Notifications
You must be signed in to change notification settings - Fork 214
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
Fix display #88
Fix display #88
Conversation
@@ -0,0 +1,182 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too many license headers in this file. Could you please keep only one original which is further below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really, I think derivative work is covered by the top license and not licensed to the ASF...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments inline
* probably not really used. | ||
* | ||
* @author Jason van Zyl | ||
* @since 3.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* @since 3.1.0 |
I guess we can remove @since
here
|
||
/** | ||
* Adapt an SLF4J logger to a Plexus logger, ignoring Plexus logger API parts that are not classical and | ||
* probably not really used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* probably not really used. | |
* probably not really used. | |
* <p> | |
* Adapted from * Adapted from https://github.com/apache/maven/blob/maven-3.6.3/maven-embedder/src/main/java/org/apache/maven/cli/logging/Slf4jLogger.java | |
@@ -0,0 +1,133 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issues as with Slf4jLogger.java: licenses, since and link to original
No description provided.