dweiss commented on a change in pull request #1830:
URL: https://github.com/apache/lucene-solr/pull/1830#discussion_r483969650
##########
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:
Doesn't the logger carry the information which project this is? This
should be sufficient to locate the file in question? I don't mind changing it
to an absolute path, but I think project + path should be enough to locate
something?
----------------------------------------------------------------
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
