borinquenkid commented on code in PR #15550:
URL: https://github.com/apache/grails-core/pull/15550#discussion_r3035630860
##########
build-logic/plugins/src/main/groovy/org/apache/grails/buildsrc/GrailsTestPlugin.groovy:
##########
@@ -50,9 +50,10 @@ class GrailsTestPlugin implements Plugin<Project> {
slurper.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd";,
false)
slurper.setFeature("http://xml.org/sax/features/namespaces";,
false)
- project.subprojects.each { subproject ->
- def testResultsDir =
subproject.layout.buildDirectory.dir("test-results").get().asFile
- if (testResultsDir.exists()) {
+ def processedDirs = [] as Set
+
Review Comment:
This one aggregates test failures across all modules into a single
.gitignored markdown file. If one runs it as a developer from root it can
pickup multiple regressions and tackle them without having to rerun again.
There is another feature that does that for the three linting checks. None
of them are part of the build process.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
