Skip to content

Blog2#3

Open
marturi wants to merge 33 commits intomasterfrom
BLOG2
Open

Blog2#3
marturi wants to merge 33 commits intomasterfrom
BLOG2

Conversation

@marturi
Copy link
Owner

@marturi marturi commented Oct 22, 2016

No description provided.

@@ -15,4 +15,8 @@ public String toto() {
public void uncoveredMethod() {
System.out.println(foo());
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MAJOR Replace this usage of System.out or System.err by a logger. rule

}

public void uncoveredMethod2() {
System.out.println(foo());
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MAJOR Replace this usage of System.out or System.err by a logger. rule

@marturi
Copy link
Owner Author

marturi commented Oct 22, 2016

SonarQube analysis reported 2 issues

  • MAJOR 2 major

Watch the comments in this conversation to review them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant