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
          11 kB
          Martin Heinzerling
        2. chart_full_range.png
          8 kB
          Martin Heinzerling
        3. chart_multiple.png
          11 kB
          Martin Heinzerling
        4. chart_nozoom.png
          8 kB
          Martin Heinzerling
        5. chart_simple.png
          5 kB
          Martin Heinzerling

          Activity

          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
          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
          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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: