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

Crop unusaged whitespace in coverage graph

    Details

    • Similar Issues:

      Description

      As described in other issues for larger the impact of a code change/coverage change is pretty small an the current graph is totally useless if you want to see any fine grain changes. Zooming the graph to the relevant scale is an major improvement here.

        Attachments

        1. chart_closeup.png
          chart_closeup.png
          11 kB
        2. chart_full_range.png
          chart_full_range.png
          8 kB
        3. chart_multiple.png
          chart_multiple.png
          11 kB
        4. chart_nozoom.png
          chart_nozoom.png
          8 kB
        5. chart_simple.png
          chart_simple.png
          5 kB

          Activity

          mheinzerling Martin Heinzerling created issue -
          mheinzerling Martin Heinzerling made changes -
          Field Original Value New Value
          Assignee stephenconnolly [ stephenconnolly ] Martin Heinzerling [ mheinzerling ]
          mheinzerling Martin Heinzerling made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          mheinzerling Martin Heinzerling added a comment -

          In general fixed: https://github.com/mheinzerling/cobertura-plugin/tree/JENKINS-16038

          As this is my first Jenkins plugin I'm looking for somebody to review my changes before I create a Pull-Requests. Furthermore getting the used eclipse formatter would be nice

          Show
          mheinzerling Martin Heinzerling added a comment - In general fixed: https://github.com/mheinzerling/cobertura-plugin/tree/JENKINS-16038 As this is my first Jenkins plugin I'm looking for somebody to review my changes before I create a Pull-Requests. Furthermore getting the used eclipse formatter would be nice
          mheinzerling Martin Heinzerling made changes -
          Assignee Martin Heinzerling [ mheinzerling ] sogabe [ sogabe ]
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Seiji Sogabe
          Path:
          .gitignore
          src/main/java/hudson/plugins/cobertura/Chartable.java
          src/main/java/hudson/plugins/cobertura/CoberturaBuildAction.java
          src/main/java/hudson/plugins/cobertura/CoberturaPublisher.java
          src/main/java/hudson/plugins/cobertura/CoverageChart.java
          src/main/java/hudson/plugins/cobertura/targets/CoverageResult.java
          src/main/resources/hudson/plugins/cobertura/CoberturaPublisher/config.jelly
          src/main/resources/hudson/plugins/cobertura/CoberturaPublisher/config.properties
          src/main/resources/hudson/plugins/cobertura/CoberturaPublisher/config_de.properties
          src/test/java/hudson/plugins/cobertura/ChartRun.java
          src/test/java/hudson/plugins/cobertura/ChartTest.java
          src/test/java/hudson/plugins/cobertura/CoberturaPublisherTest.java
          src/test/java/hudson/plugins/cobertura/CoverageResultBuilder.java
          http://jenkins-ci.org/commit/cobertura-plugin/bbef9a1d62b2592537272a5019e344850f1d924b
          Log:
          [FIXED JENKINS-16038] Crop unusaged whitespace in coverage graph

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Seiji Sogabe Path: .gitignore src/main/java/hudson/plugins/cobertura/Chartable.java src/main/java/hudson/plugins/cobertura/CoberturaBuildAction.java src/main/java/hudson/plugins/cobertura/CoberturaPublisher.java src/main/java/hudson/plugins/cobertura/CoverageChart.java src/main/java/hudson/plugins/cobertura/targets/CoverageResult.java src/main/resources/hudson/plugins/cobertura/CoberturaPublisher/config.jelly src/main/resources/hudson/plugins/cobertura/CoberturaPublisher/config.properties src/main/resources/hudson/plugins/cobertura/CoberturaPublisher/config_de.properties src/test/java/hudson/plugins/cobertura/ChartRun.java src/test/java/hudson/plugins/cobertura/ChartTest.java src/test/java/hudson/plugins/cobertura/CoberturaPublisherTest.java src/test/java/hudson/plugins/cobertura/CoverageResultBuilder.java http://jenkins-ci.org/commit/cobertura-plugin/bbef9a1d62b2592537272a5019e344850f1d924b Log: [FIXED JENKINS-16038] Crop unusaged whitespace in coverage graph
          scm_issue_link SCM/JIRA link daemon made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          mheinzerling Martin Heinzerling added a comment -

          Actually some review and comments would be nice, as this is my first contribution. I'm planning to fix some other issue here too, and I would prefer starting with the "best practice" right away

          Show
          mheinzerling Martin Heinzerling added a comment - Actually some review and comments would be nice, as this is my first contribution. I'm planning to fix some other issue here too, and I would prefer starting with the "best practice" right away
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 146840 ] JNJira + In-Review [ 192109 ]

            People

            • Assignee:
              sogabe sogabe
              Reporter:
              mheinzerling Martin Heinzerling
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: