kriegaex commented on code in PR #19:
URL:
https://github.com/apache/maven-reporting-api/pull/19#discussion_r1379470012
##########
src/main/java/org/apache/maven/reporting/MavenReport.java:
##########
@@ -84,14 +86,17 @@ public interface MavenReport {
String getDescription(Locale locale);
/**
- * Set a new output directory. Useful for staging.
+ * Set a new shared base report output directory. This directory may
contain output of other
Review Comment:
@elharo: It adds the notion that not necesarily all reports end up directly
in that directory, but that they can also end up in subdirectory. A base
directory is the flipside of the medal regarding the term subdirectory. Of
course, without the "base" prefix, that would still hold true, but like I said
in an earlier comment: Maximally terse descriptions are not always ideal from a
reader's perspective. BTW, the reader inevitably knows way less about the
meaning of parameters than the provider. She is also unlikely to read all
documentation, but only as little as possible to solve her problem or implement
her use case. I.e., she does not have the full context. And even if she does,
she is unlikely to remember all of it.
--
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
