Skip to content

ci: remove SonarQube scan from CI - #2494

Merged
skudasov merged 2 commits into
mainfrom
remove-sonarqube-ci
Oct 26, 2026
Merged

skudasov merged 2 commits into
mainfrom
remove-sonarqube-ci

Conversation

@skudasov

Copy link
Copy Markdown
Contributor

In scope of https://smartcontract-it.atlassian.net/browse/DX-5526 we are sunsetting SonarQube.

Removing the jobs that run sonarsource/sonarqube-scan-action or the shared
smartcontractkit/.github ci-sonarqube-go / ci-sonarqube-ts actions.

The SonarScanner CLI downloaded by these actions bundles a Java runtime that
still trusts the compromised eMudhra roots, and no fixed release is
available yet:

Before merging: if a SonarQube check is a required status check in branch protection or rulesets, remove it there, or this PR (and later ones) will block on it.

…are sunsetting SonarQube.

Removing the jobs that run sonarsource/sonarqube-scan-action or the shared
smartcontractkit/.github ci-sonarqube-go / ci-sonarqube-ts actions.

The SonarScanner CLI downloaded by these actions bundles a Java runtime that
still trusts the compromised eMudhra roots, and no fixed release is
available yet:
- SonarSource/sonarqube-scan-action#287
- SonarSource/sonar-scanner-cli#312
@skudasov
skudasov requested a review from a team as a code owner October 25, 2026 13:18
@github-actions

Copy link
Copy Markdown
Contributor

👋 skudasov, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@skudasov
skudasov enabled auto-merge October 26, 2026 21:16
@skudasov
skudasov added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit 7aa1615 Oct 26, 2026
48 checks passed
@skudasov
skudasov deleted the remove-sonarqube-ci branch October 26, 2026 21:45
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