SonarQube Runner
  1. SonarQube Runner
  2. SONARUNNER-39

If an aggregator module defines "sonar.sources" and has such a folder, a warning should be raised

    Details

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

      Description

      The WARNING message should be:

      /!\ A multi-module project can't have source folders, so "{path}" won't be used for the analysis. If you wan't to analyse files of this folder, you should create another sub-module and move then inside it.
      

        Activity

        Hide
        Julien HENRY added a comment - - edited

        On Sonar side it seems to be perfectly supported to have an aggregator project (ie with sub modules) also having sources. But as discussed we don't see any real life use case so we will not support this situation and consider sonar.sources defined on an aggregator as a settings to be inherited by sub modules. In case there is actually a folder matching sonar.sources in the aggregator location it could mean that user did a mistake so we will raise a warning.

        Show
        Julien HENRY added a comment - - edited On Sonar side it seems to be perfectly supported to have an aggregator project (ie with sub modules) also having sources. But as discussed we don't see any real life use case so we will not support this situation and consider sonar.sources defined on an aggregator as a settings to be inherited by sub modules. In case there is actually a folder matching sonar.sources in the aggregator location it could mean that user did a mistake so we will raise a warning.
        Hide
        Fabrice Bellingard added a comment -

        Manually tested.

        Show
        Fabrice Bellingard added a comment - Manually tested.
        Hide
        Julien HENRY added a comment -

        IT added.

        Show
        Julien HENRY added a comment - IT added.

          People

          • Assignee:
            Julien HENRY
            Reporter:
            Fabrice Bellingard
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: