You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/03/22 10:04:14 UTC

[GitHub] [camel] orpiske commented on a change in pull request #7245: (chores) cleanups for camel-report-maven-plugin

orpiske commented on a change in pull request #7245:
URL: https://github.com/apache/camel/pull/7245#discussion_r831988684



##########
File path: catalog/camel-report-maven-plugin/src/main/java/org/apache/camel/maven/ValidateMojo.java
##########
@@ -746,6 +691,68 @@ private int validateEndpointPairs(List<CamelEndpointDetails> endpoints, String s
         return errors;
     }
 
+    private StringBuilder buildValidationSuccessMessage(String str, String className, String lineNumber, String methodName, String fileName, String uri) {
+        StringBuilder sb = new StringBuilder();
+        sb.append(str);
+        if (className != null && lineNumber != null) {
+            // this is from java code
+            sb.append(className);
+            if (methodName != null) {
+                sb.append(".").append(methodName);

Review comment:
       Let's leave this for a subsequent cleanup. For now, I'd prefer to keep the changes to those that are easily/safely doable. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org