Re: [PR] rework documentation for new JDK-specific goals [maven-toolchains-plugin]

[via GitHub]

olamy commented on code in PR #32:
URL: 
https://github.com/apache/maven-toolchains-plugin/pull/32#discussion_r1564609182


##########
src/main/java/org/apache/maven/plugins/toolchain/ToolchainMojo.java:
##########
@@ -62,9 +62,9 @@ public class ToolchainMojo extends AbstractMojo {
     /**
      * Toolchains requirements, specified by one
      * <pre>  &lt;toolchain-type&gt;
-     *    &lt;param&gt;expected value&lt;/param&gt;
-     *    ...
-     *  &lt;/toolchain-type&gt;</pre>
+     *     &lt;param&gt;expected value&lt;/param&gt;
+     *     ...
+     *   &lt;/toolchain-type&gt;</pre>

Review Comment:
   what about something such 
   ```
        * <pre>{@code  <toolchain-type>
        *     <param>expected value</param>
        *     ...
        *   </toolchain-type>
        * }</pre>
        *
   ```
   I find more human readable compared to the escaped values. Should work for 
the goal page generation. 



-- 
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