gnodet commented on code in PR #1595:
URL: https://github.com/apache/maven/pull/1595#discussion_r1672188272
##########
maven-api-impl/src/main/java/org/apache/maven/internal/impl/model/MavenBuildTimestamp.java:
##########
@@ -70,7 +67,6 @@ public MavenBuildTimestamp(Instant time, String
timestampFormat) {
}
SimpleDateFormat dateFormat = new SimpleDateFormat(timestampFormat);
dateFormat.setCalendar(new GregorianCalendar());
- dateFormat.setTimeZone(DEFAULT_BUILD_TIME_ZONE);
Review Comment:
Why is that a change ? We added the timezone offset in the description
instead of using `Z`, so we need to use the current time zone, else, the
formatted time will be wrong I think.
--
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
