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

Make JacocoPublisher a SimpleBuildStep

    XMLWordPrintable

Details

    Description

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

      Attachments

        Issue Links

          Activity

            jglick Jesse Glick created issue -
            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.
            janek Jan Molak added a comment -

            +1

            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 ]
            yuri Yuri R added a comment -

            +1

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

            any update on this?

            jmvanryn John VanRyn added a comment - any update on this?

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

            mheinzerling Martin Heinzerling added a comment - jmvanryn , please check the current master of the plugin. Feel free to leave a comment if you find any issues while testing.

            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)

            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 ]

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

            Integrated in 2.0.2.

            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 -
            Resolution Fixed [ 1 ]
            Status In Progress [ 3 ] Resolved [ 5 ]
            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 ]

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

            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 ]

            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?

            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?

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

            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?

            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.

            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.

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

            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

            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

            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

            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

            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

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

              Dates

                Created:
                Updated:
                Resolved: