Bukama commented on code in PR #59:
URL:
https://github.com/apache/maven-reporting-impl/pull/59#discussion_r1957073810
##########
src/main/java/org/apache/maven/reporting/AbstractMavenReport.java:
##########
@@ -413,7 +408,7 @@ protected String getOutputEncoding() {
}
/**
- * Gets the locale
+ * Gets the locale.
Review Comment:
> No idea. That's not something this PR changed.
Well, I could argue it is, because the PR is named "clean up Javadoc" and
with the inline return statement you could eliminate the "useless" double "Gets
the locale." line. But I leave it up to you ;)
--
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
