Hi Vincent,
Thanks for the review :)
I'll post your comments on jira and revise the docs.
Odea
Vincent Siveton wrote:
Hi Maria,
Thanks for your work! Here is my comments:
usage.html
note => Bold font
add a link to the Jetty server: http://jetty.mortbay.org/jetty/index.html
sitedescriptor.html
"For more references of the site descriptor, here's a link
http://maven.apache.org/site.html"; should be in the introduction.
http://maven.apache.org/site.html is out of date thus to specify it.
(IMHO we need a new page for this, maybe something like
http://maven.apache.org/ref/current/maven-model/maven.html)
"Site Descriptor" part should be renamed to "Publish Date"
templatefile.html
add a link to Velocity: http://jakarta.apache.org/velocity/
moduleexcludes.html
empty <excludeModules/>
creatingskins.html
add an example or a link to
http://svn.apache.org/viewvc/maven/skins/trunk/maven-classic-skin/
General comment
i18n link is missing.
Moreover, i18n.xml should be updated with the new bundle.
Cheers,
Vincent
2006/7/19, Maria Odea Ching <[EMAIL PROTECTED]>:
Hi Everyone,
The Site plugin documentation is now ready for review.
A staging site is available at
http://people.apache.org/~oching/maven-site-plugin
<http://people.apache.org/%7Eoching/maven-site-plugin>.
Thanks,
Odea
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
