Skip to content

Remove SonarCloud CI integration - #56

Merged
AriusII merged 1 commit into
mainfrom
chore/remove-sonar
Sep 21, 2026
Merged

AriusII merged 1 commit into
mainfrom
chore/remove-sonar

Conversation

@AriusII

@AriusII AriusII commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Outcome

Removes the complete SonarQube Cloud integration so PRs and main no longer schedule a Sonar analysis or wait on a Sonar quality gate.

Technical scope

  • Deletes the reusable .github/workflows/sonar.yml workflow, including scanner installation, Sonar token use, coverage collection, analysis upload, and quality-gate wait.
  • Removes the sonar job, its secret dependency, and its conditional execution from both PR and main orchestration workflows.
  • Keeps the non-Sonar CI path unchanged: locked restore, Release build with warnings as errors, Microsoft Testing Platform tests and TRX, package/API validation, isolated package and template smoke tests, Native AOT probe, validation gate, and PR workflow linting.
  • No Sonar references, badges, MSBuild properties, package references, scripts, or documentation entries existed outside these workflows; none are left behind.

Validation

Check Result
git diff --check Passed
actionlint 1.7.12 downloaded with the repository-pinned SHA-256 Passed for all remaining workflows
Case-insensitive workspace scan for sonar / sonarcloud (excluding Git metadata and build output) No matches
Workflow job inspection Only CI, Lint workflows, Validate packages, template, and AOT graph, and Gate remain

No production code, package metadata, test configuration, or CI validation step was changed.

Summary by CodeRabbit

  • Chores
    • Removed automated SonarQube analysis from the main branch and pull-request CI workflows.
    • Removed the dedicated SonarQube analysis workflow, including its build, test coverage, and report-upload steps.
    • Core continuous integration and workflow-linting checks remain unchanged.

@coderabbitai

coderabbitai Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 33e396ec-359c-49c1-b9f5-bbc6feca039a

📥 Commits

Reviewing files that changed from the base of the PR and between afc0978 and 63679e8.

📒 Files selected for processing (3)
  • .github/workflows/main-ci.yml
  • .github/workflows/pull-request-ci.yml
  • .github/workflows/sonar.yml
💤 Files with no reviewable changes (3)
  • .github/workflows/main-ci.yml
  • .github/workflows/sonar.yml
  • .github/workflows/pull-request-ci.yml

Limit details: You’ve used all 10 included reviews currently available.


📝 Walkthrough

Walkthrough

The pull request removes SonarQube Cloud jobs from the main and pull-request CI workflows and deletes the reusable SonarQube Cloud workflow.

Changes

Sonar workflow removal

Layer / File(s) Summary
Remove Sonar workflow invocations
.github/workflows/main-ci.yml, .github/workflows/pull-request-ci.yml
The main and pull-request CI workflows no longer run the sonar job or pass SONAR_TOKEN.
Delete reusable Sonar workflow
.github/workflows/sonar.yml
The reusable workflow for SonarQube Cloud analysis, builds, tests, coverage validation, and artifact upload is deleted.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removal of the repository's SonarCloud CI integration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@AriusII
AriusII merged commit a552913 into main Sep 21, 2026
4 checks passed
@AriusII
AriusII deleted the chore/remove-sonar branch September 21, 2026 21:16
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.

1 participant