olamy commented on code in PR #3261:
URL: https://github.com/apache/maven-surefire/pull/3261#discussion_r2850050050
##########
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java:
##########
@@ -120,6 +121,12 @@ public class StatelessXmlReporter implements
StatelessReportEventListener<Wrappe
private final boolean enablePropertiesElement;
+ private final boolean reportTestTimestamp;
+
+ /**
+ * @deprecated Prefer adding a new constructor that accepts a
configuration object, e.g.
Review Comment:
seriously that's a lot discussion for something so much minor.
we are talking about something deprecated or not and for a class which migh
not be used at all externally....
so let's move forward there is nothing blocking at all here.
@elharo maybe focus on the new feature provided here rather than such
insignificant details
--
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]
