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

Incorrect warning level detection: Sets all warnings to level "medium"

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • violations-plugin
    • None
    • Ubuntu

      Using PHP_CodeSniffer to generate a checkstyle.xml report.

      That report is well understood by the Checkstyle plugin but not by Violations checkstyle: It sets all warnings to the "m" column (medium).

      I joined a generated "checkstyle.xml".

        1. checkstyle.xml
          136 kB
        2. screen-shot.png
          screen-shot.png
          106 kB

          [JENKINS-7308] Incorrect warning level detection: Sets all warnings to level "medium"

          wernight added a comment -

          Seems it's not only for checkstyle, but for all captured and processed XML.

          wernight added a comment - Seems it's not only for checkstyle, but for all captured and processed XML.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Peter Reilly
          Path:
          src/main/java/hudson/plugins/violations/types/checkstyle/CheckstyleParser.java
          http://jenkins-ci.org/commit/violations-plugin/63db9149b82de5c69eea349176132849ebebba6c
          Log:
          JENKINS-7308: use different mappings for checkstyle violations

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Peter Reilly Path: src/main/java/hudson/plugins/violations/types/checkstyle/CheckstyleParser.java http://jenkins-ci.org/commit/violations-plugin/63db9149b82de5c69eea349176132849ebebba6c Log: JENKINS-7308 : use different mappings for checkstyle violations

          evernat added a comment -

          It is committed (and released), so I suppose that this is fixed.

          evernat added a comment - It is committed (and released), so I suppose that this is fixed.

            peterkittreilly peterkittreilly
            wernight wernight
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: