uschindler commented on a change in pull request #1830:
URL: https://github.com/apache/lucene-solr/pull/1830#discussion_r483950589



##########
File path: gradle/validation/validate-source-patterns.gradle
##########
@@ -181,8 +252,13 @@ class ValidateSourcePatternsTask extends DefaultTask {
       }
     }
 
+    ProgressLogger progress = progressLoggerFactory.newOperation(this.class)
+    progress.start(this.name, this.name)
+
     sourceFiles.each{ f ->
-      logger.debug('Scanning source file: {}', f);
+      progress.progress("Scanning ${f.name}")
+      logger.info('Scanning source file: {}', f);

Review comment:
       Nevertheless, I'd change ours back to debug (it's really only for 
debugging). If a violation happens, filename is printed anyways.
   
   But we should change this now to not use relative paths anymore, because 
it's hard to figure out where file is located. I can take care of that.




----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to