ammachado commented on PR #1670: URL: https://github.com/apache/maven-mvnd/pull/1670#issuecomment-5152256643
Thanks for the re-review! Addressed all 3 remaining findings in 9822b4ca: 1. `BannedSkipFilter` now flushes on `CANCEL_BUILD` and `BUILD_EXCEPTION`, matching the existing `BUILD_FINISHED` behavior — no more silently dropped cosmetic reactor lines on cancelled/failed builds. 2. Replaced the stale `PROJECT_TEST_PROGRESS` TODO in `Message.java` with a description of the implemented feature. 3. `TestState.displayName()` now falls back to `"(unknown)"` instead of returning `null` when both `testClass` and `testMethod` are absent, so `flakyDetail()` can't NPE. -- 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]
