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.

Sign up for GitHub

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

Jump to bottom

CQI-10: use sonarcloud instead of sonarqube #24

Merged

dszafranek merged 5 commits into master from feature/CQI-10

Nov 16, 2023

Contributor

olewandowski1 commented Nov 6, 2023 •

edited

Loading

Within the context of this PR, I've implemented a GitHub workflow for static code analysis using SonarCloud. Additionally, I removed the SonarQube stage from the Jenkins pipeline.

EDIT: I had to correct the runApiHtmlConverter in package.json. Incorrect paths.

NOTE: The SonarCloud analysis was successfully initiated; however, it was unable to complete due to failures in the integration test suite. As a consequence, the jacocoTestReport.xml could not be generated, leading to a build failure.

What went wrong:
Execution failed for task ':integrationTest'.
> There were failing tests. See the report at: file:///app/build/reports/integrationTest/index.html


          CQI-10: use sonarcloud instead of sonarqube

a4ee96e

olewandowski1 requested a review from dszafranek

November 6, 2023 08:04

olewandowski1 added 4 commits

November 8, 2023 09:46


          CQI-10: add test coverage


          CQI-10: fix perms issue

9ee35e7


          CQI-10: perms fix

a348376


          CQI-10: add perms

311b30b

sonarcloud bot commented Nov 8, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

olewandowski1 marked this pull request as ready for review

November 9, 2023 12:08

dszafranek approved these changes

View reviewed changes

dszafranek merged commit 2dcde76 into master

3 checks passed

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

Labels

None yet