Aggregator for task list

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.6
  • Fix Version/s: 1.6
  • Labels:
    None
  • Environment:
    Add aggregator for maven task list plugin.

Description

This patch suggest to add 3 aggregators:
1) Classes tasks
2) Methods tasks
3) All tasks

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
pkernevez added a comment -

The jelly scripts and new plugin.properties

Show
pkernevez added a comment - The jelly scripts and new plugin.properties
Hide
Permalink
Vincent Massol added a comment -

Hi Kerny,

Thanks for your contribution. I would be happy to apply it. However if you wish me to apply it quickly you would also need to provide patches for:

  • documentation (changes.xml, aggregators.xml, etc)
  • functional tests. They are located in src/test/plugin-test and they need to be modified to prove that the new aggregators work.

Otherwise I would have to do this work myself and I'm not yet sure when I'll find the time to do so...

Thanks!

Show
Vincent Massol added a comment - Hi Kerny, Thanks for your contribution. I would be happy to apply it. However if you wish me to apply it quickly you would also need to provide patches for:
  • documentation (changes.xml, aggregators.xml, etc)
  • functional tests. They are located in src/test/plugin-test and they need to be modified to prove that the new aggregators work.
Otherwise I would have to do this work myself and I'm not yet sure when I'll find the time to do so... Thanks!
Hide
Permalink
pkernevez added a comment -

New patch with all modifications (changes.xml, aggregators,...)

Show
pkernevez added a comment - New patch with all modifications (changes.xml, aggregators,...)
Hide
Permalink
pkernevez added a comment -

I attach a new patch with the required data.

Show
pkernevez added a comment - I attach a new patch with the required data.
Hide
Permalink
pkernevez added a comment -

The last path with tests.

Show
pkernevez added a comment - The last path with tests.
Hide
Permalink
Vincent Massol added a comment -

Applied. Thanks for this perfect patch!

Show
Vincent Massol added a comment - Applied. Thanks for this perfect patch!

People

  • Assignee:
    Vincent Massol
    Reporter:
    pkernevez
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: