gnodet commented on PR #13233: URL: https://github.com/apache/maven/pull/13233#issuecomment-5772614394
Fixed in f784a25567: moved the skipped-phases check before `getMojoDescriptor()` in the concurrent `plan()` method. When a phase is explicitly declared on the `<execution>`, we can determine whether it is skipped without loading the plugin descriptor — this makes the `verify(pluginManager, never()).getMojoDescriptor(...)` assertion in `BuildPlanExecutorTest.skippedPhasesMojoIsNotAddedToPlan()` actually pass (the test was added in 2ce1f45e9f but the implementation hadn't been updated to match the stricter assertion). -- 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]
