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

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

    • 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".

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

          wernight created issue -
          wernight made changes -
          Attachment New: checkstyle.xml [ 19860 ]
          Description Original: 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).
          New: 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".
          Summary Original: Checkstyle setting all warning levels to medium New: Checkstyle's warning level incorrectly detected (sets all warnings to level "medium")
          wernight made changes -
          Attachment New: screen-shot.png [ 19861 ]
          wernight made changes -
          Summary Original: Checkstyle's warning level incorrectly detected (sets all warnings to level "medium") New: 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.

          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.
          evernat made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 137407 ] New: JNJira + In-Review [ 187493 ]

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

              Created:
              Updated:
              Resolved: