gnodet commented on code in PR #2278:
URL: https://github.com/apache/maven/pull/2278#discussion_r2082324824
##########
api/maven-api-core/src/main/java/org/apache/maven/api/SourceRoot.java:
##########
@@ -37,11 +37,23 @@ public interface SourceRoot {
* The path is relative to the <abbr>POM</abbr> file.
*
* <h4>Default implementation</h4>
- * The default value is <code>src/{@linkplain #scope() scope}/{@linkplain
#language() language}</code>
- * as a relative path. Implementation classes may override this default
with an absolute path instead.
+ * The default value depends on whether a {@linkplain #module() module
name} is specified in this source root:
+ * <ul>
+ * <li>
+ * If no module name, then the default directory is
Review Comment:
I'm not sure that's the right angle to look at.
If a given project has multiple modules, those modules _kinda_ act as
subprojects, so it makes sense to group their files together, rather than
splitting main / tests. In that case, each test directory is testing the code
of the module, so having them close to each other makes more sense to me.
--
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
