desruisseaux commented on code in PR #2180: URL: https://github.com/apache/maven/pull/2180#discussion_r2011809001
########## api/maven-api-core/src/main/java/org/apache/maven/api/SourceRoot.java: ########## @@ -26,13 +26,23 @@ /** * A root directory of source files. * The sources may be Java main classes, test classes, resources or anything else identified by the scope. + * + * <h2>Default values</h2> + * The default implementation of all methods in this interface match the default values + * documented in {@code maven.mdo} (the <abbr>POM</abbr> model). Review Comment: True, I will do that. Actually, I didn't realized that these generated classes where in public API. Since they are, shouldn't we go ahead with the use of `record` before GA, or is it too late in 4.0 release candidates for that? -- 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