-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat(ci): Configure SonarCloud Analysis on CI #6432
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just some initial feedback based on my assumptions so forgive my ignorance if i am not following things correctly
Kudos, SonarCloud Quality Gate passed! |
Description
This PR updates the SonarCloud analysis to be run after the unit tests complete for PR analysis. You can see the successful SonarCloud run and results in the body of this PR.
Further configuration will be needed because we haven't had a successful run on
main
.Screenshots/Recordings
If applicable, add screenshots and/or recordings to visualize the before and after of your change
Issue
Progresses #???
Checklist