david vicente wrote:
I would like to perform the first release of the Dashboard Maven Plugin.
0, not using it. Notes:
The POM contains no <scm> element, causing a bad "Source Repository" report.
The POM defines remote repositories. If these are not required, they
could be deleted to avoid their maintenance.
Also, I believe you can delete <project>/<organization> from the POM
which should be inherited from the Mojo parent.
The <project>/<url> element in the POM erroneously points to the Mojo
homepage, not the plugin homepage.
"mvn dependency:analyze" shows some undeclared dependencies.
The site descriptor can be trimmed down, i.e. some things are inherited [0].
The plugin declares a dependency on plexus-utils:1.3 but has only a
prerequisite on Maven 2.0.4. If the plugin doesn't work with
plexus-utils:1.1, you should update the prerequisite to 2.0.6 [1].
Checkstyle/PMD reports a few violations, mostly formatting and javadoc.
But there are also things like missing hashCode() implementation to
accompany equals() which might be worth to have a look it.
Some POM snippets in the documentation are a little hard to read because
of the mixed usage of tabs and spaces for their indentation.
Bonne chance for the release, hope you have less problems with the
Release Plugin this time ;-)
Benjamin
[0] http://mojo.codehaus.org/development/guidelines.html
[1] http://maven.apache.org/guides/plugin/guide-java-plugin-development.html
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
