Tibor17 commented on PR #517: URL: https://github.com/apache/maven-surefire/pull/517#issuecomment-1094347033
@sbabcoc @SykApps @slachiewicz Now the OOP looks better in the latest commit. WDYT? We can better concentrate on the details implemented in both methods. @slachiewicz This would help us when we would perform the removal of TestNG 5.x and we can much better merge the logic in a new objective design across the version 6.x and 7.x. Currently, some options, like `setJunit`, are lost and that's we did not controll Configuration in pull requests, we only accepted and that's the penalty. -- 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