Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix: #201 could not decorate gitlab merge request #214

Conversation

PiekJ
Copy link
Contributor

@PiekJ PiekJ commented Jul 20, 2020

Enables GitLab merge request to be decorated. This PR fixes issue #201 for SonarQube 8.2+.

Copy link
Owner

@mc1arke mc1arke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your Pull Request. Please squash your commits into a single commit for this feature and ensure the commit message contains any details relevant for review and subsequent understanding of the change when looking in the git log

@PiekJ PiekJ force-pushed the bugfix/201-could-not-decorate-gitlab-merge-request branch 5 times, most recently from 884b625 to 4fc9569 Compare July 23, 2020 08:57
@PiekJ
Copy link
Contributor Author

PiekJ commented Jul 23, 2020

I resolved your comment about the star import and squashed my commits.

Edit: I'll rebase the other PR's when this one got merged.

@PiekJ PiekJ force-pushed the bugfix/201-could-not-decorate-gitlab-merge-request branch from 4fc9569 to db04a2c Compare July 25, 2020 12:18
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

92.7% 92.7% Coverage
0.0% 0.0% Duplication

@PiekJ PiekJ force-pushed the bugfix/201-could-not-decorate-gitlab-merge-request branch from db04a2c to e01a8e1 Compare July 27, 2020 07:07
Fixed setting the project id on sonarqube project pull decorate option when selecting GitLab.
When project id is set use that instead of the sonarqube scanner property.
Fixed always evaluating fetching sonarqube scanner property for instance url.
@PiekJ PiekJ force-pushed the bugfix/201-could-not-decorate-gitlab-merge-request branch from e01a8e1 to 6030eed Compare August 3, 2020 08:22
@PiekJ
Copy link
Contributor Author

PiekJ commented Aug 3, 2020

I resolved the suggestions.

@mc1arke mc1arke merged commit 0e8cbea into mc1arke:master Aug 3, 2020
@mc1arke
Copy link
Owner

mc1arke commented Aug 3, 2020

Merged. Thanks for the contribution!

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.

2 participants