SonarQube is now a real reviewer! SonarQube Stash (BitBucket) plugin is a pull-request decorator which allows to integrate SonarQube violations directly into your pull-request.
After every run, in addition of the diff view, you may access to an overview of your SQ analysis:
- Git client to checkout the code
- Maven 3.0.5+
- JDK 1.7 or 1.8
- SonarQube 4.5.4 (LTS) or later
- Stash (BitBucket) REST API 1.0 (3.x, 4.x)
Note: these are the versions where the plugin has been tested. Other versions may or may not work, YMMV.
This command generates a jar file:
mvn clean package
Just copy the sonar-stash-plugin jar file to the plugin folder of the expected SonarQube server and restart the SonarQube server. For instance, on Linux platform:
cp target/sonar-stash-plugin-1.0.jar $SONARQUBE_HOME/extensions/plugins
Go to Stash general settings screen on SonarQube server to fill:
Stash base URL (sonar.stash.url): To define Stash instance.
Stash base user (sonar.stash.login): To define user to push violations on Stash pull-request. User must have REPO_READ permission for the repository. Please notice Stash password needs to be provided to sonar-runner through sonar.stash.password on the commandline.
Stash coverage severity (sonar.stash.coverage.severity.threshold): If the contribution reduced the coverage on a file, the plugin generates an issue with the following severity.
Stash issue threshold (sonar.stash.issue.threshold): To limit the number of issue pushed to Stash.
Stash timeout (sonar.stash.timeout): To timeout when Stash Rest api does not replied with expected.
Stash reviewer approval (sonar.stash.reviewer.approval): SonarQube is able to approve the pull-request if there is no new issue introduced by the change. By default, this feature is deactivated: if activated, Stash base user must have REPO_WRITE permission for the repositories.
Stash tasks severity threshold (sonar.stash.task.issue.severity.threshold): SonarQube is able to create tasks for all issues with a severity higher to the threshold. By default, this feature is deactivated (threshold: NONE).
To activate the plugin, just add the following options to the SonarQube launcher (for instance with sonar-runner):
Before SonarQube 5.2:
sonar-runner -Dsonar.analysis.mode=incremental \
-Dsonar.stash.notification -Dsonar.stash.project=<PROJECT> -Dsonar.stash.repository=<REPO> \
-Dsonar.stash.pullrequest.id=<PR_ID> -Dsonar.stash.password=<STASH_PASSWORD>...
For SonarQube 5.2+:
sonar-runner -Dsonar.analysis.mode=preview \
-Dsonar.stash.notification=true -Dsonar.stash.project=<PROJECT> -Dsonar.stash.repository=<REPO> \
-Dsonar.stash.pullrequest.id=<PR_ID> -Dsonar.stash.password=<STASH_PASSWORD>...
If needed, you can reset comments published during the previous SonarQube analysis of your pull-request. Please add sonar.stash.comments.reset option to your SonarQube analysis. Please notice only comments linked to the sonar.stash.login user will be deleted. This reset will be the first action performed by the plugin.
sonar-runner -Dsonar.analysis.mode=incremental -Dsonar.stash.notification -Dsonar.stash.comments.reset -Dsonar.stash.project=<PROJECT> -Dsonar.stash.repository=<REPO> -Dsonar.stash.pullrequest.id=<PR_ID> -Dsonar.stash.password=<STASH_PASSWORD>...
The plugin can push into the pull-request the coverage computed with the help of the Unit Tests. This feature is based on "line coverage" metric, which compares the "uncovered lines" with the "lines to cover" on a file.
To activate this feature,
sonar-runner -Dsonar.branch=<BRANCH_TO_BE_COMPARED> -Dsonar.stash.notification -Dsonar.stash.project=<PROJECT> -Dsonar.stash.repository=<REPO> -Dsonar.stash.pullrequest.id=<PR_ID> -Dsonar.stash.password=<STASH_PASSWORD> -Dsonar.scanAllFiles...
Be careful, with SonarQube 4.5.6, this feature does not work with incremental mode.
The plugin can also read the password from an environment variable.
This is configured by setting sonar.stash.password.variable
to the name of
the environment variable to read.
The prevents the password from leaking into the process table.
-
Before developing a major feature please open a ticket and announce it. Maybe the maintainers have strong opinions or useful hints about it.
-
Add unit and for major features integration tests.
-
Use the Google Java Style Guide for new development.