I agree with copying the default value into the end of the description.
The reason it isn't in the table as a column is that it is too wide,
usually.
I would leave the expression as just the expression in the latter part
as it is now. We've already agreed this is unclear and will deprecate it
in 2.1 in favour of:
- default-value="..." (always the default, can contain expressions)
- propertyKey="maven.test.skip" (instead of expression
${maven.test.skip}, for allowing properties to configure it from the
pom, settings and command line. Perhaps needs a different name).
I guess we could hack it such that expressions that aren't project.* or
settings.* are treated that way in the document, but let's think some
more about that.
As for the other suggests - I also agree we should have a little summary
at the top that shows the mojo level metadata (@phase, @goal, @execute,
@requiresDependencyResolution).
I also agree we should link to the Javadoc of complex types. However,
this is non-trivial, so let's enter it as a new JIRA issue for now.
Cheers,
Brett
On 29/06/2006 12:53 PM, Edwin Punzalan wrote:
+1 for moving the default value. A lot of the other comments want the
default value at the top of the page. Showing the default value in a
sentence is nice too.
It just somehow becomes complicated because expressions functions
somewhat like a default value, too. Any suggestions on how to handle that?
Mark Hobson wrote:
Definitely a big improvement :)
I agree that it'd be nice to see the default value for optional
parameters in the table. Perhaps append it to the description, e.g.
appendAssemblyId boolean Set to false to exclude the
assembly id
from the assembly final name. (Default value is true)
Also it'd be nice at least to link complex object types to their
javadoc, e.g. MavenArchiveConfiguration in maven-jar-plugin. An
example XML block from introspecting the bean would be even better ;)
Cheers,
Mark
On 28/06/06, Brett Porter <[EMAIL PROTECTED]> wrote:
Hi,
Edwin has come up with this:
http://jira.codehaus.org/secure/attachment/21233/assembly-mojo.html
Any feedback before I apply it?
- Brett
--
Brett Porter <[EMAIL PROTECTED]>
Apache Maven - http://maven.apache.org/
Better Builds with Maven - http://library.mergere.com/
---------------------------------------------------------------------
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]
--
Brett Porter <[EMAIL PROTECTED]>
Apache Maven - http://maven.apache.org/
Better Builds with Maven - http://library.mergere.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
