Conversation
| @@ -16,5 +16,16 @@ public void coveredByIntegrationTest() { | |||
| public void notCovered() { | |||
| System.out.println("notCovered"); | |||
There was a problem hiding this comment.
Replace this usage of System.out or System.err by a logger. 
| package example; | ||
|
|
||
| public class HelloWorld { | ||
| public class HelloWorld implements Cloneable { |
There was a problem hiding this comment.
Add a "clone()" method to this class.
|
|
||
|
|
||
| public void morenotCovered() { | ||
| System.out.println("morenotCovered"); |
There was a problem hiding this comment.
Replace this usage of System.out or System.err by a logger.
Define a constant instead of duplicating this literal "morenotCovered" 3 times.
| public class HelloWorld implements Cloneable { | ||
|
|
||
| public void coveredByUnitTest() { | ||
| System.out.println("coveredByUnitTest1"); |
There was a problem hiding this comment.
Replace this usage of System.out or System.err by a logger.
| } | ||
|
|
||
| public void morenotCovered2() { | ||
| System.out.println("morenotCovered"); |
There was a problem hiding this comment.
Replace this usage of System.out or System.err by a logger.
| } | ||
|
|
||
| public void morenotCovered3() { | ||
| System.out.println("morenotCovered"); |
There was a problem hiding this comment.
Replace this usage of System.out or System.err by a logger.
|
SonarQube analysis reported 11 issues
Watch the comments in this conversation to review them. 4 extra issuesNote: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:
|
1 participant
No description provided.