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

Trigger Sonarcloud analysis from GHA #143

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Conversation

kwin
Copy link
Contributor

@kwin kwin commented Jun 3, 2024

Only generate coverage info conditionally (with profile "code-coverage")

@kwin kwin force-pushed the feature/fix-code-coverage branch 5 times, most recently from 36f63d7 to 6f6aaad Compare June 3, 2024 14:07
@kwin
Copy link
Contributor Author

kwin commented Jun 3, 2024

Looks good now, coverage not yet available (https://sonarcloud.io/summary/new_code?id=eclipse-sisu_sisu.inject&pullRequest=143), because this is an incremental analysis only. It needs to be executed on main branch first.

Only generate coverage info conditionally (with profile "code-coverage")
This closes #140
@kwin kwin force-pushed the feature/fix-code-coverage branch from 6f6aaad to d7544b8 Compare June 3, 2024 14:49
@kwin kwin marked this pull request as ready for review June 3, 2024 14:49
@kwin kwin requested a review from cstamas June 3, 2024 14:49
Copy link

sonarcloud bot commented Jun 3, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@kwin kwin merged commit 43a200d into main Jun 3, 2024
13 checks passed
@kwin kwin deleted the feature/fix-code-coverage branch June 3, 2024 14:52
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