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

          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!

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

              Created:
              Updated:
              Resolved: