Uploaded image for project: 'SonarQube'
  1. SonarQube
  2. SONAR-9917

Add the branch when a notification is triggered on a short living or a long living branch

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 6.7
    • Component/s: Branch & PR, Notifications
    • Labels:

      Description

      Currently the emails sent on a short-living branch does not contain the branch in the body of the email :

      When there is a change in issue :

      sources/Person.js
      Rule: "=+" should not be used instead of "+="
      Message: Was "+=" meant instead?
      
      Resolution:  (was FIXED)
      Status: REOPENED (was RESOLVED)
      
      More details at: http://localhost:9000/project/issues?id=org.sonarqube%3Ajavascript-lcov-sq-scanner2&issues=AV7np_Q6RB9nKOR1DGpj&open=AV7np_Q6RB9nKOR1DGpj&branch=test1
      

      On creation of new issue, it's not that readable (between parenthesis) :

      Project: JavaScript :: Coverage LCOV :: SonarQube Scanner (test1)
      Version: 1.0
      
      1 new issue (new debt: 2min)
      
          Type
              Bug: 1    Vulnerability: 0    Code Smell: 0
      
          Rules
              "=+" should not be used instead of "+=" (js): 1
      
          Most impacted files
              Person.js: 1
      
      More details at: http://localhost:9000/project/issues?id=org.sonarqube%3Ajavascript-lcov-sq-scanner2&branch=test1&createdAt=2017-10-04T15%3A58%3A42%2B0200
      

      We should have it at the beginning of the email :

      Project: JavaScript :: Coverage LCOV :: SonarQube Scanner
      Branch:  test1
      Version: 1.0
      ...
      sources/Person.js
      Rule: "=+" should not be used instead of "+="
      Message: Was "+=" meant instead?
      
      Resolution:  (was FIXED)
      Status: REOPENED (was RESOLVED)
      
      More details at: http://localhost:9000/project/issues?id=org.sonarqube%3Ajavascript-lcov-sq-scanner2&issues=AV7np_Q6RB9nKOR1DGpj&open=AV7np_Q6RB9nKOR1DGpj&branch=test1
      

      and

      Project: JavaScript :: Coverage LCOV :: SonarQube Scanner
      Branch: test1
      Version: 1.0
      
      1 new issue (new debt: 2min)
      
          Type
              Bug: 1    Vulnerability: 0    Code Smell: 0
      
          Rules
              "=+" should not be used instead of "+=" (js): 1
      
          Most impacted files
              Person.js: 1
      
      More details at: http://localhost:9000/project/issues?id=org.sonarqube%3Ajavascript-lcov-sq-scanner2&branch=test1&createdAt=2017-10-04T15%3A58%3A42%2B0200
      

        Attachments

          Activity

            People

            • Assignee:
              julien.lancelot Julien Lancelot
              Reporter:
              eric.hartmann Eric Hartmann
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved: