shoemoney commented on PR #12832: URL: https://github.com/apache/maven/pull/12832#issuecomment-5465172738
Thanks for the review and the approval, @gnodet — and good catch on the #12911 overlap. CI was red on `MojoExecutorForkedExecutionTest.executeForkedExecutionsThrowsDescriptiveExceptionForUnknownProject`: the test that landed with #12911 asserts the message contains `not in the reactor`, and my rewritten messages had dropped that phrase. Pushed 9057215c to restore the original wording and append the parallel-build / extension context to it rather than replacing it, so both the assertion and the enriched diagnostics hold. Verified locally on maven-core: 620 tests, 0 failures (was 1), Checkstyle 0 violations, Spotless clean. -- 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]
