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

MSBuild parser ignores StyleCop warnings

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Minor
    • Resolution: Fixed
    • warnings-plugin
    • None
    • Hudson 1.384, warnings plugin 3.11, Visual Studio 2010, StyleCop, Windows XP

    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

          drulli Ulli Hafner added a comment -

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

          drulli Ulli Hafner added a comment - Integrated in Hudson Plug-ins (Compile) #80 [FIXED JENKINS-8347] Improved regular expression to support style cop warnings.
          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?

          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?
          drulli Ulli Hafner added a comment -

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

          drulli Ulli Hafner added a comment - Which count do you mean? The count in the violations plgu-in or in the warnings plug-in?
          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.

          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

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

            Dates

              Created:
              Updated:
              Resolved: