-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18473/
-----------------------------------------------------------

Review request for streams.


Repository: streams


Description
-------

Having cobertura reports are handy for a project to allow developers to focus 
unit testing efforts.

As part of this, I updated the poms to allow for running "mvn site".  As part 
of site, cobertura reports will be generated.  I've also configured cobertura 
so it can be executed via "mvn cobertura:cobertura" which will be useful for 
the builds.apache.org environment.

The changes to wrappers/pom.xml were necessary otherwise the maven site plugin 
threw errors when generating anchors for those elements.

WARNING: Running mvn site for this project with maven 3.1.x will fail.  Only 
Maven 3.0.x can be used.  If we wish to support 3.1.x we'll need to update 
several plugin versions.


Diffs
-----

  http://svn.apache.org/repos/asf/incubator/streams/trunk/pom.xml 1571357 
  http://svn.apache.org/repos/asf/incubator/streams/trunk/poms/wrappers/pom.xml 
1571357 

Diff: https://reviews.apache.org/r/18473/diff/


Testing
-------

Ran "mvn site" locally and verified that cobertura reports were present.

Ran "mvn clean verify" locally and verified that cobertura reports were not 
present.

Ran "mvn clean install cobertura:cobertura" on my personal Jenkins system and 
verified cobertura reports were present in a format that Jenkins could report.


Thanks,

Stanton Sievers

Reply via email to