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

#674: Add support for Sonarqube 9.7 #699

Merged
merged 1 commit into from
Dec 11, 2022
Merged

Conversation

mc1arke
Copy link
Owner

@mc1arke mc1arke commented Dec 10, 2022

Sonarqube has removed the use of the same external component key and differing database IDs for different branches on the same projects and now use different IDs in all cases. The pull request web service endpoints have also been removed from community edition. To allow the plugin to work with the new version of Sonarqube, the component key generation for branches has been modified to save branch DTOs whenever a new branch is created, and to remove the conditions around re-using the same branch if the target branch details matched an existing branch. The Pull Request endpoint actions have been copied from the old community edition sources, and tidied up to use a cleaner abstraction model. As the front-end only shows branch features if the implementation of BranchFeatureExtension returns the name 'branch-support', the CommunityBranchFeatureExtension has been altered to follow this requirement, and an additional MonoRepoFeature has been implemented to allow the mono-repo switches to be shown against the front-end.

Includes the migration of any altered unit tests to JUnit 5.

@mc1arke mc1arke mentioned this pull request Dec 10, 2022
@mc1arke mc1arke force-pushed the feature/sonarqube-9_7-support branch from 662c1d4 to f616341 Compare December 11, 2022 12:51
Sonarqube has removed the use of the same external component key and
differing database IDs for different branches on the same projects and
now use different IDs in all cases. The pull request web service
endpoints have also been removed from community edition. To allow the
plugin to work with the new version of Sonarqube, the component key
generation for branches has been modified to save branch DTOs whenever a
new branch is created, and to remove the conditions around re-using the
same branch if the target branch details matched an existing branch. The
Pull Request endpoint actions have been copied from the old community
edition sources, and tidied up to use a cleaner abstraction model. As
the front-end only shows branch features if the implementation of
`BranchFeatureExtension` returns the name 'branch-support', the
`CommunityBranchFeatureExtension` has been altered to follow this
requirement, and an additional `MonoRepoFeature` has been implemented to
allow the mono-repo switches to be shown against the front-end.

Includes the migration of any altered unit tests to JUnit 5.
@mc1arke mc1arke force-pushed the feature/sonarqube-9_7-support branch from f616341 to d9a0180 Compare December 11, 2022 15:25
@sonarcloud
Copy link

sonarcloud bot commented Dec 11, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

98.0% 98.0% Coverage
1.3% 1.3% Duplication

@mc1arke mc1arke merged commit a1f28e5 into master Dec 11, 2022
@mc1arke mc1arke deleted the feature/sonarqube-9_7-support branch December 11, 2022 15:33
@RedlineTriad
Copy link

@mc1arke When do you plan to create a release with these changes?
Would be nice to be able to update Sonar to latest.

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.

3 participants