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

Crop unusaged whitespace in coverage graph

XMLWordPrintable

    • Icon: New Feature New Feature
    • Resolution: Fixed
    • Icon: Minor Minor
    • cobertura-plugin
    • None

      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.

        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

          [JENKINS-16038] Crop unusaged whitespace in coverage graph

          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

          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

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

          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

          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

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

              Created:
              Updated:
              Resolved: