Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-27120

Make JacocoPublisher a SimpleBuildStep

    Details

    • Similar Issues:

      Description

      See the guide to allowing this publisher to be used from a Workflow.

        Attachments

          Issue Links

            Activity

            jglick Jesse Glick created issue -
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: COMPATIBILITY.md http://jenkins-ci.org/commit/workflow-plugin/2ec399aa1d54432b22fbd5052b24d824da4670d3 Log: JENKINS-27120 JENKINS-27121 JENKINS-26953 JENKINS-27123 JENKINS-26432 Noting.
            Hide
            janek Jan Molak added a comment -

            +1

            Show
            janek Jan Molak added a comment - +1
            asotobueno Alex Soto made changes -
            Field Original Value New Value
            Assignee Ognjen Bubalo [ ognjenb ] Alex Soto [ asotobueno ]
            recena Manuel Recena Soto made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            recena Manuel Recena Soto made changes -
            Remote Link This issue links to "PR (Web Link)" [ 13705 ]
            Hide
            yuri Yuri R added a comment -

            +1

            Show
            yuri Yuri R added a comment - +1
            Show
            asotobueno Alex Soto added a comment - https://github.com/jenkinsci/jacoco-plugin/pull/63
            hrmpw Patrick Wolf made changes -
            Epic Link JENKINS-34657 [ 170293 ]
            Hide
            jmvanryn John VanRyn added a comment -

            any update on this?

            Show
            jmvanryn John VanRyn added a comment - any update on this?
            Hide
            mheinzerling Martin Heinzerling added a comment -

            John VanRyn, please check the current master of the plugin. Feel free to leave a comment if you find any issues while testing.

            Show
            mheinzerling Martin Heinzerling added a comment - John VanRyn , please check the current master of the plugin. Feel free to leave a comment if you find any issues while testing.
            Hide
            szhem Sergey Zhemzhitsky added a comment -

            Just tested the master branch and it seems to work pretty well, except for the trend graphs are not shown in the main job view (JENKINS-31202)

            Show
            szhem Sergey Zhemzhitsky added a comment - Just tested the master branch and it seems to work pretty well, except for the trend graphs are not shown in the main job view ( JENKINS-31202 )
            centic centic made changes -
            Link This issue depends on JENKINS-31202 [ JENKINS-31202 ]
            Hide
            mheinzerling Martin Heinzerling added a comment -

            centic My test were also successful. "Go" from my point of view.

            Integrated in 2.0.2.

            Show
            mheinzerling Martin Heinzerling added a comment - centic My test were also successful. "Go" from my point of view. Integrated in 2.0.2.
            mheinzerling Martin Heinzerling made changes -
            Status In Progress [ 3 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            mheinzerling Martin Heinzerling made changes -
            Link This issue depends on JENKINS-31202 [ JENKINS-31202 ]
            mheinzerling Martin Heinzerling made changes -
            Link This issue is related to JENKINS-36536 [ JENKINS-36536 ]
            Hide
            mheinzerling Martin Heinzerling added a comment -

            Created new issue for the trend graph: JENKINS-36536 (an started already)

            Show
            mheinzerling Martin Heinzerling added a comment - Created new issue for the trend graph: JENKINS-36536 (an started already)
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 161316 ] JNJira + In-Review [ 196714 ]
            Hide
            brianstclair Brian St.Clair added a comment -

            I'd like to get this update. I looked through the GitHub repo here https://github.com/jenkinsci/jacoco-plugin. Is that the right repo? Apologies, but I can't find the commit/label/release to pull that includes this change. Is it in there somewhere?

            Show
            brianstclair Brian St.Clair added a comment - I'd like to get this update. I looked through the GitHub repo here https://github.com/jenkinsci/jacoco-plugin . Is that the right repo? Apologies, but I can't find the commit/label/release to pull that includes this change. Is it in there somewhere?
            Hide
            brianstclair Brian St.Clair added a comment -

            I just found this PR: https://github.com/jenkinsci/jacoco-plugin/pull/66 but I don't see the commit in master?

            Show
            brianstclair Brian St.Clair added a comment - I just found this PR: https://github.com/jenkinsci/jacoco-plugin/pull/66 but I don't see the commit in master?
            Hide
            mheinzerling Martin Heinzerling added a comment -

            This commits are in the master, it just a kind of messed up in the history. Either use the current master for the minimal pipeline support or if mheinzerling_2.1.0 a try, where I already add some improvements.

            Show
            mheinzerling Martin Heinzerling added a comment - This commits are in the master, it just a kind of messed up in the history. Either use the current master for the minimal pipeline support or if mheinzerling_2.1.0 a try, where I already add some improvements.
            Hide
            brianstclair Brian St.Clair added a comment -

            Thanks! I'll give mheinzerling_2.1.0 a shot and let you know if I run into any issues. Much appreciated.

            Show
            brianstclair Brian St.Clair added a comment - Thanks! I'll give mheinzerling_2.1.0 a shot and let you know if I run into any issues. Much appreciated.
            abayer Andrew Bayer made changes -
            Labels workflow pipeline workflow
            abayer Andrew Bayer made changes -
            Labels pipeline workflow pipeline
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            COMPATIBILITY.md
            http://jenkins-ci.org/commit/pipeline-plugin/63791640268fa092d44c66f2bc98f9351049d1c8
            Log:
            Merge pull request #410 from netmikey/patch-1

            JENKINS-27120 Mark JaCoCoPublisher as compatible

            Compare: https://github.com/jenkinsci/pipeline-plugin/compare/f80ee9fafc7a...63791640268f

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: COMPATIBILITY.md http://jenkins-ci.org/commit/pipeline-plugin/63791640268fa092d44c66f2bc98f9351049d1c8 Log: Merge pull request #410 from netmikey/patch-1 JENKINS-27120 Mark JaCoCoPublisher as compatible Compare: https://github.com/jenkinsci/pipeline-plugin/compare/f80ee9fafc7a...63791640268f
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Alex Soto
            Path:
            pom.xml
            src/main/java/hudson/plugins/jacoco/JacocoBuildAction.java
            src/main/java/hudson/plugins/jacoco/JacocoPublisher.java
            src/main/java/hudson/plugins/jacoco/JacocoReportDir.java
            src/main/java/hudson/plugins/jacoco/model/CoverageObject.java
            src/main/java/hudson/plugins/jacoco/report/AbstractReport.java
            src/main/java/hudson/plugins/jacoco/report/CoverageReport.java
            src/test/java/hudson/plugins/jacoco/report/AbstractReportTest.java
            src/test/java/hudson/plugins/jacoco/report/CoverageReportTest.java
            src/test/java/hudson/plugins/jacococoveragecolumn/JaCoCoColumnTest.java
            http://jenkins-ci.org/commit/jacoco-plugin/ce1f7322f1462ba35c2f7cecc1556d2456066c64
            Log:
            [FIXED JENKINS-27120] Adding Workflow support for JaCoCo publisher

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Alex Soto Path: pom.xml src/main/java/hudson/plugins/jacoco/JacocoBuildAction.java src/main/java/hudson/plugins/jacoco/JacocoPublisher.java src/main/java/hudson/plugins/jacoco/JacocoReportDir.java src/main/java/hudson/plugins/jacoco/model/CoverageObject.java src/main/java/hudson/plugins/jacoco/report/AbstractReport.java src/main/java/hudson/plugins/jacoco/report/CoverageReport.java src/test/java/hudson/plugins/jacoco/report/AbstractReportTest.java src/test/java/hudson/plugins/jacoco/report/CoverageReportTest.java src/test/java/hudson/plugins/jacococoveragecolumn/JaCoCoColumnTest.java http://jenkins-ci.org/commit/jacoco-plugin/ce1f7322f1462ba35c2f7cecc1556d2456066c64 Log: [FIXED JENKINS-27120] Adding Workflow support for JaCoCo publisher

              People

              • Assignee:
                asotobueno Alex Soto
                Reporter:
                jglick Jesse Glick
              • Votes:
                15 Vote for this issue
                Watchers:
                27 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: