Skip to content

Feature/sonar 6.7+ support#1

Open
brianbaker wants to merge 3 commits intocignium:masterfrom
markitondemand:feature/sonar-6.7-support
Open

Feature/sonar 6.7+ support#1
brianbaker wants to merge 3 commits intocignium:masterfrom
markitondemand:feature/sonar-6.7-support

Conversation

@brianbaker
Copy link
Copy Markdown

The current version of the plugin uses the deprecated sonar.branch parameter. This PR updates that param to use sonar.branch.name. See https://docs.sonarqube.org/display/PLUG/Branch+Plugin

@brianbaker
Copy link
Copy Markdown
Author

I had to remove the sonar.project.branch param entirely as Sonar requires that the first scan of a project not supply sonar.branch.name. After that first scan, sonar.branch.name can be supplied. My recommendation is to remove the parameter entirely. Its easy enough to supply it in the additional args. Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant