<div dir="ltr"><div>Such a feature would indeed be great. We already use it on the ROS 2 build farm (which is just a set of hand crafted jobs right now).</div><div><br></div>Installing the <a href="https://wiki.jenkins-ci.org/display/JENKINS/Cobertura+Plugin">https://wiki.jenkins-ci.org/display/JENKINS/Cobertura+Plugin</a> on the build farm and adding the configuration to look e.g. for `catkin_workspace/build*/*/coverage.xml` should be easy. But that means that the tests need to generate the coverage result files. And as far as I know they currently don't. So I think the generation of the coverage file(s) has to happen first.<div><br></div><div>It would be great if someone would look into this and push on adding this feature. I am happy to help with answering questions etc.<br><div><br></div><div>Thanks,</div><div>- Dirk</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 17, 2016 at 11:22 AM, Bence Magyar <span dir="ltr"><<a href="mailto:bence.magyar.robotics@gmail.com" target="_blank">bence.magyar.robotics@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Great! It's good to enforce some quality measures on released code. Are you planning to add the test coverage visualisation(no enforcement though) to Jenkins too? It would be useful for projects that are heavily tested such as ros_control.</p>
<p dir="ltr">Cheers,<br>
Bence</p>
<div class="gmail_quote"><div><div class="h5">On 17 May 2016 19:06, "Dirk Thomas via ros-release" <<a href="mailto:ros-release@lists.ros.org" target="_blank">ros-release@lists.ros.org</a>> wrote:<br type="attribution"></div></div><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">The new build farm was already detecting compiler warnings but didn't do anything about them - until now. If a devel job (same for PR jobs which only have a different trigger mechanism) has compiler warnings the job will become unstable. This also results in a notification being sent to the maintainers, the same as if a job fails.<div><br></div><div>Since we don't want to change the behavior of the numerous devel jobs of already released distributions this has only been enabled for Kinetic. Of the 154 existing devel jobs in Kinetic only 18 have compiler warnings but are "green" otherwise. We think it is reasonable for the maintainers to address those.</div><div><br></div><div>If it happens to be a repository you maintain you might have already or will in the near future receive an email from Jenkins about this. Please consider taking the short amount of time to address the compiler warning in your code. The greater benefit is that all the other packages will benefit from this stricter check and can get notified properly when a regression indicated by a compiler warning happens.</div><div><br></div><div>Thank you,</div><div>- Dirk</div></div>
<br></div></div>_______________________________________________<br>
ros-release mailing list<br>
<a href="mailto:ros-release@lists.ros.org" target="_blank">ros-release@lists.ros.org</a><br>
<a href="http://lists.ros.org/mailman/listinfo/ros-release" rel="noreferrer" target="_blank">http://lists.ros.org/mailman/listinfo/ros-release</a><br>
<br></blockquote></div>
</blockquote></div><br></div>