Tibor17 commented on PR #517: URL: https://github.com/apache/maven-surefire/pull/517#issuecomment-1097198159
@sbabcoc We are not doing the refactoring due to reflection. Reflection is a minimum problem for the developer. The main problem in this Configuration is the fact that we relied on the quality of external contributors that this part of code was out of our control and the quality was really low, even buggy. This PR is non technical fix in real. We will continue with @slawekjaranowski after this PR and we will serve to our contributors and users with higher quality. The next is the removal of TestNG 5.x and the reason behind it is the fact that 5.x does not work on JDK 15+ and the version is over 10 years old. We will remove Surefire provider `surefire-junit4` because it has the same features as `surefire-junit47`, we save 25% of our cost, we speed up the CI, and we guess that the users won't use obsolete and narrow range of JUnit [4.0, 4.7) and mostly they use 3.8.2 (surefire-junit3), 4.8.2 (surefire-junit47), 4.12 and 4.13.x, we wil have an unsupported gap 4.0 - 4.7. Altogether, this this activity is much more than a little tiny refactoring. -- 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
