streams-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stanton Sievers" <siever...@gmail.com>
Subject Review Request 18473: Enabling site building and cobertura reporting
Date Tue, 25 Feb 2014 17:08:00 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message