Skip to content

ci: replace Codecov with SonarCloud coverage#175

Merged
BKDDFS merged 3 commits into
mainfrom
claude/interesting-ride-036abe
Jun 6, 2026
Merged

ci: replace Codecov with SonarCloud coverage#175
BKDDFS merged 3 commits into
mainfrom
claude/interesting-ride-036abe

Conversation

@BKDDFS

@BKDDFS BKDDFS commented Jun 6, 2026

Copy link
Copy Markdown
Owner

What & why

Replaces Codecov with SonarCloud coverage (same setup as omniviser/omniray).

Changes

  • .github/workflows/sonar.yml (new) β€” dedicated SonarCloud job: fork guard, fetch-depth: 0, Python 3.13, uv, tests with --cov-report=xml, SonarSource/sonarqube-scan-action (SHA-pinned v8.1.0).
  • run_tests.yml β€” removed Codecov upload step; test job only runs tests with --cov-fail-under=100.
  • sonar-project.properties (new) β€” org=bkddfs, key=BKDDFS_PerfectFrameAI, sources=perfectframe, coverage from coverage.xml.
  • README.md β€” Codecov badge replaced with SonarCloud coverage badge.

Manual setup (outside repo, done)

  1. Generated new SONAR_TOKEN and added it to both Actions and Dependabot secrets.
  2. Disabled Automatic Analysis on SonarCloud (conflicts with CI-based scan).
  3. Optional: remove the unused CODECOV_TOKEN secret.

BKDDFS and others added 3 commits June 6, 2026 14:00
- add .github/workflows/sonar.yml running tests with coverage and
  SonarCloud scan (fork guard, fetch-depth 0)
- drop Codecov upload step from run_tests.yml
- add sonar-project.properties (org BKDDFS, key BKDDFS_PerfectFrameAI)
- swap Codecov badge for SonarCloud coverage badge in README
Org slug is lowercase 'bkddfs' (per prior SonarCloud setup), not 'BKDDFS'
β€” wrong casing caused HTTP 403 from api.sonarcloud.io.
@BKDDFS BKDDFS merged commit f50c529 into main Jun 6, 2026
10 of 12 checks passed
@BKDDFS BKDDFS deleted the claude/interesting-ride-036abe branch June 6, 2026 12:41
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