David wrote:

I'd like to release dbunit v1.0-beta-2

+0, not using it. Notes:

The "type" parameter in OperationMojo has
  @parameter expression="type"
Note the missing ${...} delimiters in the expression. Similiar for the parameter "transaction".

The POM is missing the <scm> element.

The site descriptor can be trimmed down, i.e. name="Mojo", <banner*>, ${reports} and the <links> to Maven/Mojo can be removed since inherited.

The FAQ is empty. IMHO, having empty docs (likely just because of docck:check) is pointless. As a curious user, I see its link in the menu and click it just to end up with nothing. Always reminds me of Nelson's "Ha Ha!" ;-). If there's no real FAQ, I suggest to just kill the page.

The POM snippet on the usage.apt does not have a <version> element for the plugin, promoting bad practice.

Not relevant for the release but more for the build in general: The plugin's IT is unreliable. For instance, src/examples/pom.xml has the plugin version not locked and hence simply used version 1.0-beta-1 during my local build instead of the actual project JAR.


Benjamin

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to