|
||||||||
|
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators. For more information on JIRA, see: http://www.atlassian.com/software/jira |
||||||||
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
Hi @Louis, all.
I've merged the pull request #2 [1] that adds the same feature (from what I could tell by reading the code).
@Louis, unfortunately your PR has changes in the pom.xml, as well as major changes in Ccm.java and Metric.java, for what I believe was the IDE mixing up tabs/spaces. So I couldn't merge it :o(
However, your pull request and the pull request #2 look very alike. However the #2 has none tests. While your Pull Request does! Yay! So I'm massaging your tests a little bit to adapt to the current master/origin branch, and will commit tonight.
Could someone lend me a hand testing, please? I'll generate an alpha version and will post it here. I haven't used the plug-in in a while, so I'm having troubles setting up my Eclipse+Maven environment in my Windows box.
Thank you all and sorry for taking so long to look at this issue.
[1] https://github.com/jenkinsci/ccm-plugin/pull/2