Complete settings
All threads resolved!
Please also review the wiki section https://wikis.uni-paderborn.de/pgandi/Routing_Protocol_Evaluations#Database_Application which discuss why these settings were chosen.
Review !86 (merged) first.
Edited by Ghost User
Merge request reports
Activity
SonarQube analysis reported 7 issues
-
⛔ 1 blocker -
🚫 3 critical -
🔽 3 minor
-
⛔ Add some tests to this class.📘 -
🚫 Make this class "Serializable".📘 -
🚫 Make this class "Serializable".📘 -
🚫 Make this class "Serializable".📘 -
🔽 Drop this interface in favor of "java.util.function.BiFunction<Collection,List,Collection>".📘 -
🔽 Add a 'package-info.java' file to document the 'choosers' package📘 -
🔽 Add a 'package-info.java' file to document the 'prioritizers' package📘
-
SonarQube analysis reported 7 issues
-
⛔ 1 blocker -
🚫 3 critical -
🔽 3 minor
-
⛔ Add some tests to this class.📘 -
🚫 Make this class "Serializable".📘 -
🚫 Make this class "Serializable".📘 -
🚫 Make this class "Serializable".📘 -
🔽 Drop this interface in favor of "java.util.function.BiFunction<Collection,List,Collection>".📘 -
🔽 Add a 'package-info.java' file to document the 'choosers' package📘 -
🔽 Add a 'package-info.java' file to document the 'prioritizers' package📘
-
SonarQube analysis reported 18 issues
-
⛔ 1 blocker -
🚫 3 critical -
⚠ 1 major -
🔽 13 minor
-
⛔ Add some tests to this class.📘 -
🚫 Make this class "Serializable".📘 -
🚫 Make this class "Serializable".📘 -
🚫 Make this class "Serializable".📘 -
⚠ Assign this magic number 2 to a well-named constant, and use the constant instead.📘 -
🔽 Drop this interface in favor of "java.util.function.BiFunction<Collection,List,Collection>".📘 -
🔽 Make this line start at column 13.📘 -
🔽 Define the locale to be used in this String operation.📘 -
🔽 Replace these toUpperCase()/toLowerCase() and equals() calls with a single equalsIgnoreCase() call.📘 -
🔽 Make this line start at column 17.📘 -
🔽 Replace these toUpperCase()/toLowerCase() and equals() calls with a single equalsIgnoreCase() call.📘 -
🔽 Make this line start at column 17.📘 -
🔽 Make this line start at column 17.📘 -
🔽 Make this line start at column 21.📘 -
🔽 Move this "else" on the same line that the previous closing curly brace.📘 -
🔽 Make this line start at column 13.📘 -
🔽 Add a 'package-info.java' file to document the 'choosers' package📘 -
🔽 Add a 'package-info.java' file to document the 'prioritizers' package📘
-
added 112 commits
-
9bab4330...3cab61ae - 111 commits from branch
master
- 41b62f1d - Merge remote-tracking branch 'remotes/origin/master' into feature-#318-complete-settings
-
9bab4330...3cab61ae - 111 commits from branch
added 73 commits
-
41b62f1d...4e3b775a - 72 commits from branch
master
- 90d2da6a - Merge remote-tracking branch 'remotes/origin/master' into feature-#318-complete-settings
-
41b62f1d...4e3b775a - 72 commits from branch
added Needs Review label
added Needs information from dev and removed Needs Review labels
added Needs information from reviewer and removed Needs information from dev labels
mentioned in commit a5b7ebd3
Please register or sign in to reply