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

Reply via email to