Tibor17 commented on PR #3200: URL: https://github.com/apache/maven-surefire/pull/3200#issuecomment-3499069429
@olamy I talked to @Frawless . We think the same, that #3194 was too eagerly pushed, Not only tests were missing but the functionality is hard to understand due to it splits in two commits. Therefore we tried to have only one simple commit. Hopefully, there are only trivial changes: 3x _dependabot_ and one _Javadoc_, so we can replay them (in IntelliJ IDEA by cherry pick) again after we are done. The class in @Frawless 's change will be touched later and therefore it is important to have this feature in a consistent state, means one commit. It will be easy to understand that feature change if it is in one commit and not two (even far away in history). -- 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]
