-
Notifications
You must be signed in to change notification settings - Fork 515
ci: remove sonarqube #15557
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
ci: remove sonarqube #15557
Conversation
this service will be decomissioned shortly
🚀 Benchmarks reportTo see the full report comment with |
fr4nc1sc0-r4m0n
left a comment
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.
LGTM
mrodm
left a comment
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.
LGTM!
Looking at the active backport branches, I think this change should also be backported to these branches that have been updated lately:
|
@v1v I think this PR could be merged, since those tests failing in CI are already reported as flaky tests here: |
jsoriano
left a comment
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.
@v1v do we have an alternative service for code coverage?
I'm not aware of anything. We stopped using codecov longtime ago - maybe it could be revisited |
|
@Mergifyio backport backport-security_detection_engine-9.0 backport-security_detection_engine-8.19 backport-security_detection_engine-8.18 |
✅ Backports have been created
|
(cherry picked from commit 2ab655d)
(cherry picked from commit 2ab655d)
(cherry picked from commit 2ab655d)
this service will be decomissioned shortly
Proposed commit message
ci: remove sonarqube
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots