gnodet opened a new pull request, #13182: URL: https://github.com/apache/maven/pull/13182
Forward-port of review findings from #13173 to `master`. Applied to both `compat/maven-embedder` and `impl/maven-cli`: - **Local StringBuilder**: move the `StringBuilder` out of the `ReactorSummaryRequest` record and allocate it locally inside `logReactorSummaryGroup`. The record is a pure data-holder; carrying mutable state in it is a latent correctness trap. - **UNKNOWN group=2**: assign `group = 2` to the `else` branch so any future `BuildSummary` subclass is rendered last at error level rather than silently grouped with SKIPPED modules. -- 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]
