hboutemy commented on code in PR #1466: URL: https://github.com/apache/maven/pull/1466#discussion_r1566385346
########## maven-core/src/site/apt/artifact-handlers.apt: ########## @@ -35,7 +35,7 @@ Default Artifact Handlers Reference *--------------------+---------------+------------+------------+-----------+---------------------+-----------------------+ || type || classifier || extension || packaging || language || added to classpath || includesDependencies || *--------------------+---------------+------------+------------+-----------+---------------------+-----------------------+ -| <<<pom>>> | | <= type> | <= type> | none | | | +| <<<pom>>> | | <= type>, Note: this differs for POM files outside the repository which usually use <<<xml>>> as filesystem extension. | <= type> | none | | | Review Comment: sorry, we had so many discussions that I don't see how to tell. as said by Tamasz, there is a mix In addition, such a text will break the table display. eventually, if you absolutely want to write something, you can add a note at the end of the page that could say something like: > Dependency type <<\<pom>>> has an extension <<<.pom>>> because POM is stored in repositories as <<<artifactId-version.pom>>>. > During development, as stored in source control, the file is by default named <<<pom.xml>>>" I don't see how useful it is, but it says what you seem to need to say and does not break the dependency type table -- 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