Analysis Bootstrapper for Visual Studio Projects
  1. Analysis Bootstrapper for Visual Studio Projects
  2. SONARVS-3

Module exclusion don't work when the project key contains a ':' colon character (using the "safe" key strategy)

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0
    • Labels:
      None
    • Number of attachments :
      0

      Description

      According to Julien Henry, there is a little hack in the way SonarQube supports sonar.skippedModules to make it easier to exclude Maven modules, and which explains why we're having troubles getting it to work on .NET.

      If your project key is set to "ann:dotnet.test1", and your module name is "testTests", then your module key becomes [project_key]:[module_name], i.e. "ann:dotnet.test1:testTests".

      One would expect to use sonar.skippedModules=testTests to skip that specific module.

      However, because of that little hack on SonarQube, you actually need to specify everything which is after the first colon : symbol, i.e. dotnet.test1:testTests in this case.

      The best thing to do is to not use the : colon symbol in project keys, it will make your life much easier, because sonar.skippedModules=[module_name] will just work.

        Issue Links

          Activity

          Hide
          Dinesh Bolkensteyn added a comment -

          Well no the title doesn't say it all - it does not say with which strategy exclusions are broken.

          I'll have to figure it out

          Show
          Dinesh Bolkensteyn added a comment - Well no the title doesn't say it all - it does not say with which strategy exclusions are broken. I'll have to figure it out
          Hide
          Dinesh Bolkensteyn added a comment -

          Note that the "safe" mode will always be enabled in 2.2.

          Show
          Dinesh Bolkensteyn added a comment - Note that the "safe" mode will always be enabled in 2.2.
          Hide
          Dinesh Bolkensteyn added a comment -

          Should be fixed, could you please test @Ann?

          Show
          Dinesh Bolkensteyn added a comment - Should be fixed, could you please test @Ann?
          Hide
          G. Ann Campbell added a comment -

          I can't make exclusions work

          Show
          G. Ann Campbell added a comment - I can't make exclusions work
          Hide
          Dinesh Bolkensteyn added a comment -

          I've updated the ticket's description to mention everything which was discovered Ann, and I let you redo the test as discussed.

          Show
          Dinesh Bolkensteyn added a comment - I've updated the ticket's description to mention everything which was discovered Ann, and I let you redo the test as discussed.
          Hide
          G. Ann Campbell added a comment -

          manually tested

          Show
          G. Ann Campbell added a comment - manually tested

            People

            • Assignee:
              Dinesh Bolkensteyn
              Reporter:
              Alexandre Victoor
            • Votes:
              2 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: