kevinnammour commented on code in PR #3261:
URL: https://github.com/apache/maven-surefire/pull/3261#discussion_r2847403879
##########
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/DefaultStatelessReportMojoConfiguration.java:
##########
@@ -45,6 +45,7 @@ public DefaultStatelessReportMojoConfiguration(
String xsdSchemaLocation,
boolean enableOutErrElements,
boolean enablePropertiesElement,
+ boolean reportTestTimestamp,
Review Comment:
Totally agree on the principle. However, the commit history shows multiple
minor releases that added new fields to this same constructor—also a breaking
API change. On top of that, this class has been deprecated for seven years,
with no documentation pointing to the recommended replacement.
What’s your recommendation for getting this into the next milestone? I’m all
for good practices, but they’re meaningless if half the codebase is deprecated
and there’s no migration path.
--
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]
