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

PEP8 warning priority is too high

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • warnings-plugin
    • None
    • Warnings plugin 4.51

      According to the pep8 tool documentation (https://pep8.readthedocs.org/en/latest/intro.html#error-codes), the reported warnings always are errors (E) or warnings (W).
      The Pep8Parser maps this categories to the HIGH priority.

      IMO, pep8 warnings are less important than other warnings like pylint and should not use the HIGH priority but LOW for warnings (W) and NORMAL for errors (E).

      Is this change acceptable ?

      Regards.

          [JENKINS-32295] PEP8 warning priority is too high

          Ulli Hafner added a comment -

          Interested in providing a pull request? I don't use the pep parser by myself...

          Ulli Hafner added a comment - Interested in providing a pull request? I don't use the pep parser by myself...

          Jonathan Delizy added a comment -

          I have never done that before but I could try.

          Jonathan Delizy added a comment - I have never done that before but I could try.

          Ulli Hafner added a comment -

          The source code is quite simple, should not be hard to change the mapping:

          Currently there is support for different characters, I'm not sure where these are coming from... Feel free to change!

          Ulli Hafner added a comment - The source code is quite simple, should not be hard to change the mapping: Pep8Parser Pep8ParserTest Example Log for Tests Currently there is support for different characters, I'm not sure where these are coming from... Feel free to change!

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/warnings/parser/Pep8Parser.java
          src/test/java/hudson/plugins/warnings/parser/Pep8ParserTest.java
          src/test/resources/hudson/plugins/warnings/parser/pep8Test.txt
          http://jenkins-ci.org/commit/warnings-plugin/77c4b4d70b7419c923b679b61cdd182a6da99e4e
          Log:
          [FIXED JENKINS-32295] Reduce priorities of parser.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/warnings/parser/Pep8Parser.java src/test/java/hudson/plugins/warnings/parser/Pep8ParserTest.java src/test/resources/hudson/plugins/warnings/parser/pep8Test.txt http://jenkins-ci.org/commit/warnings-plugin/77c4b4d70b7419c923b679b61cdd182a6da99e4e Log: [FIXED JENKINS-32295] Reduce priorities of parser.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          warnings
          http://jenkins-ci.org/commit/analysis-suite-plugin/b0b48021263dd2d263988fe798441541967b31b9
          Log:
          [FIXED JENKINS-32295] Reduce priorities of parser.

          Compare: https://github.com/jenkinsci/analysis-suite-plugin/compare/a10d9985ed92...b0b48021263d

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: warnings http://jenkins-ci.org/commit/analysis-suite-plugin/b0b48021263dd2d263988fe798441541967b31b9 Log: [FIXED JENKINS-32295] Reduce priorities of parser. Compare: https://github.com/jenkinsci/analysis-suite-plugin/compare/a10d9985ed92...b0b48021263d

            drulli Ulli Hafner
            jonathand131 Jonathan Delizy
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: