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

warnings-ng: pylint parser should use new-style labels

XMLWordPrintable

    • warnings-ng 2.0.0, analysis-model-api 2.0.0

      Currently the category for  the builtin parser  is set to the old-style numeric warnings  ("msg_id") (W0xxx). I think it would be better to use the textual representation by default ( the "symbol"), as these are easily human-readable, while the numeric ones always require a lookup.

          [JENKINS-55340] warnings-ng: pylint parser should use new-style labels

          Björn Pedersen created issue -

          Ulli Hafner added a comment - - edited

          Can you make that more clear by providing the a full message line and the expected result? In my test case I have

          trunk/src/python/tv.py:35: [C0111, Episode] Missing docstring

          Should C0111 be replaced by Episode, or where is the textual representation? Since I don't use python at all, interested in providing a PR?

          Ulli Hafner added a comment - - edited Can you make that more clear by providing the a full message line and the expected result? In my test case I have trunk/src/python/tv.py:35: [C0111, Episode] Missing docstring Should C0111 be replaced by Episode , or where is the textual representation? Since I don't use python at all, interested in providing a PR?
          Ulli Hafner made changes -
          Labels New: help-wanted newbie-friendly
          Ulli Hafner made changes -
          Assignee Original: Ulli Hafner [ drulli ]
          Ulli Hafner made changes -
          Component/s New: analysis-model [ 23523 ]
          Component/s Original: warnings-ng-plugin [ 24526 ]

          Ulli Hafner added a comment -

          Or is this related to JENKINS-44915?

          Ulli Hafner added a comment - Or is this related to JENKINS-44915 ?

          Björn Pedersen added a comment -

          Yes , that is related.  I'll  try if I can provide a pull request for this.

          Björn Pedersen added a comment - Yes , that is related.  I'll  try if I can provide a pull request for this.

          Björn Pedersen added a comment -

          https://github.com/jenkinsci/analysis-model/pull/73

          Björn Pedersen added a comment - https://github.com/jenkinsci/analysis-model/pull/73

          Björn Pedersen added a comment -

          In your example,  Episode is the class (or function?) that is  missing the docstring. This part could be  used to calculate module/package information for python projects. But that would be a separate topic.

          Björn Pedersen added a comment - In your example,  Episode is the class (or function?) that is  missing the docstring. This part could be  used to calculate module/package information for python projects. But that would be a separate topic.
          Ulli Hafner made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Fixed but Unreleased [ 10203 ]

            Unassigned Unassigned
            pedersen Björn Pedersen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: