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

Option to auto escape characters not allowed in sonar branches

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Major Major
    • sonar-plugin
    • None

      In light of http://jira.codehaus.org/browse/SONAR-4188, there are some branch names that just aren't allowed.

      In our Sonar configuration we'd like to set the 'Branch' field to '${GIT_BRANCH}'

      Due to '/' being commonly used in git branches, we'd like to add a checkbox under the branch field called 'Escape invalid branch characters' so any invalid character is automatically changed to underscore '_'.

      We plan on submitting a patch for this - I'm hoping you will consider applying it.

          [JENKINS-21607] Option to auto escape characters not allowed in sonar branches

          pmv added a comment -

          Visual of what the option would look like

          pmv added a comment - Visual of what the option would look like

          pmv added a comment -

          Attaching patch. Let me know if you have any feedback, thanks.

          pmv added a comment - Attaching patch. Let me know if you have any feedback, thanks.

          This feature is a must to use Sonar together with Git because nearly all Git-based workflows propose a branch naming pattern like "bugfixes/bugXY" or "features/newFeatureXY" which at least have one character ('/') that is not allowed in Sonar when using "-Dsonar.branch=". Would be highly appreciated!

          Michael Rüegg added a comment - This feature is a must to use Sonar together with Git because nearly all Git-based workflows propose a branch naming pattern like "bugfixes/bugXY" or "features/newFeatureXY" which at least have one character ('/') that is not allowed in Sonar when using "-Dsonar.branch=". Would be highly appreciated!

          chirs damour added a comment -

          super +1 on this

          chirs damour added a comment - super +1 on this

            sonarteam Sonar Team
            pmv pmv
            Votes:
            2 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: