Skip to content

chore: remove SonarQube/SonarCloud integration - #2099

Open
0xCVH wants to merge 1 commit into
zkevmfrom
chore/remove-sonar
Open

chore: remove SonarQube/SonarCloud integration#2099
0xCVH wants to merge 1 commit into
zkevmfrom
chore/remove-sonar

Conversation

@0xCVH

@0xCVH 0xCVH commented Jul 10, 2026

Copy link
Copy Markdown

Part of the org-wide SonarQube/SonarCloud deprecation (consolidating on CodeQL + GitHub Advanced Security).

  • Removed the SonarCloud Scan step from .github/workflows/ci_zkevm.yml (the Test job and everything else untouched).
  • Deleted sonar-project.properties.

🤖 Generated with Claude Code

Part of the org-wide Sonar deprecation (consolidating on CodeQL + GitHub Advanced Security). Removes the SonarCloud scan step from ci_zkevm.yml and deletes sonar-project.properties.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 10, 2026 14:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the repository’s SonarQube/SonarCloud configuration as part of the org-wide deprecation in favor of CodeQL + GitHub Advanced Security.

Changes:

  • Removed the SonarCloud GitHub Actions step from the ZK EVM CI workflow.
  • Deleted the sonar-project.properties configuration file.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
sonar-project.properties Removes SonarCloud project configuration and analysis settings.
.github/workflows/ci_zkevm.yml Drops the SonarCloud Scan step while leaving the rest of the CI workflow unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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