Hi John,

Done. However it was more complex. Here's the commit message:

"
Added JUnit aggregators. Thanks to John D Taylor. John I had to change
several things from your patch:
  - remove tabs
  - added missind documentation (xdoc)
  - added missing tests
  - it was not working. More specifically it would not work if the
clover aggregators were used before the junit ones
  - the goal caused by the junit aggregators was not the correct one...
(you were calling "test" whereas it's the junit-report that must be
called)

  Please note that I had to modify both the Clover plugin and the
junit-report one. So if you wish to use the junit aggregators you need
to have the CVS versions of the these 2 plugins installed.
"

Glad you liked the dashboard plugin! 

Thanks
-Vincent

> -----Original Message-----
> From: John D Taylor (ROE) [mailto:[EMAIL PROTECTED]
> Sent: 29 November 2003 16:13
> To: [EMAIL PROTECTED]
> Cc: Maven Users List
> Subject: Dashboard question and junit aggregator
> 
> Hi Vincent,
> Thankyou for the Dashboard plugin - I've found it effective and
> straightforward to use (even if I can't get clover reports just now -
> eagerly awaiting the next release...)
> Please find attached some aggregators I have written for junit - do
with
> them what you will.
> One thing I realised while I was writing them is that when you want to
> extract several columns from one report it's essentially a copy and
paste
> job.  Do you have any plans (or is there a way I don't know of) to be
able
> to pass parameters to the jelly script so you can reuse the same
script
> with
> different column names?
> John


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to