Thanks everyone for your comments :-)
I'll post them in Jira and I'll fix the docs :-)


Odea

Vincent Siveton wrote:

Thanks Maria, here my comments:

usage.html
Should be better to create two subsections:
- Generate the report as part of Project Reports
- Generate the report as standalone

Maybe add a report screenshot

FAQ
"checkstyle.properties" or "checkstyle properties" (with space)
If the first, the following seems wrong according the doc
<configLocation>checkstyle.properties</configLocation>
http://people.apache.org/~oching/maven-checkstyle-plugin/checkstyle-mojo.html#configLocation

custom-property-expansion.html
propertiesLocation has no sample. Is is the same explained in FAQ?

multi-module-config.html
Review it as said
Replace unix commands by a directory layout:
whizbang
|-- pom.xml
`-- src
   |-- main
...

Cheers,

Vincent

2006/7/24, Maria Odea Ching <[EMAIL PROTECTED]>:

Hi Everyone,

The Checkstyle plugin documentation is now ready for review :)
A staging site is available at
http://people.apache.org/~oching/maven-checkstyle-plugin
<http://people.apache.org/%7Eoching/maven-checkstyle-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]

Reply via email to