Hi Stephen Great work on this plugin! This is a plugin that I plan to use extensively.
I've read through the docs, which there are plenty of. Always a good sign :-) There were however a couple of typos and broken links in there, which I took the liberty of fixing in SVN. I also updated to mojo-parent 18. After playing around with the plugin a bit I found the results somewhat confusing, so I started to read the goal parameter docs. There I found the source of my confusion: the allowSnapshots parameter has true as default value. In my opinion this should be set to false as default. Using snapshots is something that should be avoided, if possible. Showing snapshot versions by default therefor works against best practices and might lure users to the dark side. I'd like to move the includeProperties, excludeProperties and linkItems parameters from the Abstract Mojo to UpdatePropertiesMojo, because they are only used there. Also the parentVersion parameter should be moved to UpdateParentMojo. Having them in the Abstract Mojo makes them show up as parameters in the auto generated docs for every Mojo, see [1]. The comparisonMethod parameter is currently only used in DisplayDependencyUpdatesMojo. Shouldn't it be used in DisplayPluginUpdatesMojo as well? If not, it can be moved from the Abstract Mojo to DisplayDependencyUpdatesMojo. Finally the auto generated docs would benefit from using "default-value" in the @parameter annotations. This automatically inserts the default values into the docs. If you want me to, I can have a go at making these changes. [1] http://mojo.codehaus.org/versions-maven-plugin/display-dependency-updates-mojo.html Stephen Connolly wrote: > Folks, I've been working on the versions-maven-plugin and I think it's > ready to cut the first alpha release. > > The major changes in this release are > - Fixed MOJO-1209 (required a rewrite of the display-plugin-updates goal) > > Known issues > > - MOJO-1210 display-plugin-updates does not include lifecycle plugins > that are not defined in the pom > - MOJO-1211 display-plugin-updates does not identify the plugin > version as not being provided when derived from the super-pom > > Both of these issues will require a bit of work and I'd rather get an > alpha release out before trying to fix them. > > The new site has just been deployed here: > http://mojo.codehaus.org/versions-maven-plugin > > Snapshot have been published on > http://snapshots.repository.codehaus.org/org/codehaus/mojo/versions-maven-plugin > > [+1] release it > [0] don't care > [-1] don't release ! > > Vote will be open for 72 hours and will conclude via lazy consensus. > > Please vote :-) > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Dennis Lundberg --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
