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

MSBuild parser ignores StyleCop warnings

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: warnings-plugin
    • Labels:
      None
    • Environment:
      Hudson 1.384, warnings plugin 3.11, Visual Studio 2010, StyleCop, Windows XP
    • Similar Issues:

      Description

      We've got a Visual Studio 2010 solution with one or more projects whose build process includes calling on StyleCop, which generates warnings about people ignoring naming conventions, layout etc.

      Although these warnings fit the same "format" as standard MSBuild warnings, the plugin seems to ignore them and does not report their presence.

      e.g.

      MoqExtensions.cs(2,1): warning : SA1210: Using directives must be sorted alphabetically by the namespaces. [C:\hudsonSlave\workspace\MyProject\Source\Common.Tests.Stubs.csproj]
      MoqExtensions.cs(3,1): warning : SA1210: Using directives must be sorted alphabetically by the namespaces. [C:\hudsonSlave\workspace\MyProject\Source\Common.Tests.Stubs.csproj]
      MoqExtensions.cs(4,1): warning : SA1210: Using directives must be sorted alphabetically by the namespaces. [C:\hudsonSlave\workspace\MyProject\Source\Common.Tests.Stubs.csproj]
      MoqExtensions.cs(6,1): warning : SA1208: System using directives must be placed before all other using directives. [C:\hudsonSlave\workspace\MyProject\Source\Common.Tests.Stubs.csproj]
      MoqExtensions.cs(70,1): warning : SA1402: A C# document may only contain a single class at the root level unless all of the classes are partial and are of the same type. [C:\hudsonSlave\workspace\MyProject\Source\Common.Tests.Stubs.csproj]
      

        Attachments

          Activity

          Hide
          drulli Ulli Hafner added a comment -

          Integrated in Hudson Plug-ins (Compile) #80
          [FIXED JENKINS-8347] Improved regular expression to support style cop warnings.

          Show
          drulli Ulli Hafner added a comment - Integrated in Hudson Plug-ins (Compile) #80 [FIXED JENKINS-8347] Improved regular expression to support style cop warnings.
          Hide
          trondandersen trondandersen added a comment -

          This issue creates a side effect. When StyleCop compiler warnings are included and you also uses StyleCop for the Violation plugin, the count for each StyleCop error gets duplicated. Any thoughts regarding this? Could this be easily solved?

          Show
          trondandersen trondandersen added a comment - This issue creates a side effect. When StyleCop compiler warnings are included and you also uses StyleCop for the Violation plugin, the count for each StyleCop error gets duplicated. Any thoughts regarding this? Could this be easily solved?
          Hide
          drulli Ulli Hafner added a comment -

          Which count do you mean? The count in the violations plgu-in or in the warnings plug-in?

          Show
          drulli Ulli Hafner added a comment - Which count do you mean? The count in the violations plgu-in or in the warnings plug-in?
          Hide
          trondandersen trondandersen added a comment -

          Sorry. Forgot to mentioned that I meant the Continuous integration game plugin which will increase and decrease with two points - one for the compiler warning and another for the StyleCop error.

          Show
          trondandersen trondandersen added a comment - Sorry. Forgot to mentioned that I meant the Continuous integration game plugin which will increase and decrease with two points - one for the compiler warning and another for the StyleCop error.

            People

            • Assignee:
              drulli Ulli Hafner
              Reporter:
              pjdarton pjdarton
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: