• 5.0.0 (analysis-model and warnings-ng)

      Given the following stage definition and the attached CodeSniffer output
      I was expecting to have errors reported.
      Instead I get warnings with severity HIGH

      Changing the tool from phpCodeSniffer() to checkStyle() makes no difference

      stage('CodeSniffer') {
        steps {
          sh '''
             vendor/bin/phpcs \\
             --ignore=*/vendor/*,*/node_modules/* \\
             --extensions=php \\
             --standard=phpcs.xml \\
             --bootstrap=vendor/autoload.php \\
             --report=checkstyle \\
             --report-file=phpcs.log.xml \\
             Modules/
          '''
        }
      
        post {
          always {
            recordIssues enabledForFailure: true,
            aggregatingResults: true,
            tools: [phpCodeSniffer(pattern: 'phpcs.log.xml', reportEncoding: 'UTF-8')]
          }
        }
      }
      

          [JENKINS-56214] PHP CodeSniffer errors treated as warnings

          Cătălin Nicolescu created issue -
          Ulli Hafner made changes -
          Component/s Original: analysis-model-api-plugin [ 24221 ]
          Component/s Original: warnings-ng-plugin [ 24526 ]
          Assignee Original: Ulli Hafner [ drulli ]
          Labels New: help-wanted newbie-friendly
          Priority Original: Major [ 3 ] New: Minor [ 4 ]
          Ulli Hafner made changes -
          Epic Link New: JENKINS-56456 [ 198054 ]
          Tanja Roithmeier made changes -
          Assignee New: Tanja Roithmeier [ tanjaro ]
          Tanja Roithmeier made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Tanja Roithmeier made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Ulli Hafner made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Fixed but Unreleased [ 10203 ]
          Ulli Hafner made changes -
          Released As New: 5.0.0 (analysis-model and warnings-ng)
          Status Original: Fixed but Unreleased [ 10203 ] New: Resolved [ 5 ]

            tanjaro Tanja Roithmeier
            catalin_nicolescu Cătălin Nicolescu
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: