gnodet opened a new pull request, #13091: URL: https://github.com/apache/maven/pull/13091
Follow-up to #13055 and the review on #13067. Two improvements to `BuildPlanExecutor`: ### Javadoc clarification The `isFatal` `@return` tag now explains *why* `RuntimeException` is treated as fatal on par with `Error`: both indicate an unexpected JVM or framework state, as opposed to checked exceptions which are ordinary plugin failures that allow the reactor to continue. ### New test: soft-failure path `exceptionThrownByBuildStepIsRecordedAsBuildFailure` uses `IllegalStateException` (a `RuntimeException`, fatal path). There was no test pinning the soft-failure path — a checked exception that must *not* halt the reactor when `--fail-at-end` is in effect. The new `checkedExceptionThrownByBuildStepDoesNotHaltReactor` test fills that gap. -- 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]
