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

MsBuildParser should parse Code Analysis warnings

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Not A Defect
    • Icon: Minor Minor
    • warnings-plugin
    • None

      Running MSBuild with Code Analysis enabled outputs warnings like this:

      (RunCodeAnalysis target) ->
      c:\dev\proj\Example\MyClass.cs(10): warning CA2214: Microsoft.Usage : 'MyClass.MyClass()' contains a call chain that results in a call to a virtual method defined by the class. Review the following call stack for unintended consequences: \r [C:\dev\proj\Example\Example.csproj]

          [JENKINS-27976] MsBuildParser should parse Code Analysis warnings

          Stuart Grassie created issue -

          Ulli Hafner added a comment -

          Interested in providing a pull request (incl. a test) that improves the regular expression?

          Ulli Hafner added a comment - Interested in providing a pull request (incl. a test) that improves the regular expression?

          Stuart Grassie added a comment -

          So I went through the process of forking the repo, getting maven and eclipse configured, produced some sample output, wrote a unit test... and then discovered there isn't in fact a bug at all, it does parse the code analysis warnings correctly, and the fault was in my build process not being quite right.

          </sigh>

          Stuart Grassie added a comment - So I went through the process of forking the repo, getting maven and eclipse configured, produced some sample output, wrote a unit test... and then discovered there isn't in fact a bug at all, it does parse the code analysis warnings correctly, and the fault was in my build process not being quite right. </sigh>

          Ulli Hafner added a comment -

          Ups. At least you are now prepared for the next enhancement

          And you can still submit the additional unit test as pull request if you like!

          Ulli Hafner added a comment - Ups. At least you are now prepared for the next enhancement And you can still submit the additional unit test as pull request if you like!
          Ulli Hafner made changes -
          Resolution New: Not A Defect [ 7 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 162575 ] New: JNJira + In-Review [ 196994 ]

            drulli Ulli Hafner
            sgrassie Stuart Grassie
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: