michael-o commented on PR #670: URL: https://github.com/apache/maven-surefire/pull/670#issuecomment-1866161097
> > Please let me know whether you want to be merged. > > Yes, the problem really important fo our project. Sorry for long response, not much time in the end of year. I'll fix remarks, but now I can see one problem. > > > Also, were can I see that this applies to passed tests only? > > I'm little misunderstood issue that I linked with. For few our projects there is important to disable all output elements for all success and failure tests (how it worked in previous versions). @michael-o, what behaviour do you think is required as part of the plugin: the ability to disable output for all tests (as in this PR, my cases) or for successful tests only (as initially asked in issue)? Scratch my previous statement. Re-reviewed your PR, from a static PoV, it looks reasonable. In fact, I picked up your idea and created #702. After it is merged, I will go over to your PR. -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
