split test-with-coverage.yml workflow

[jrosental]

Summary by CodeRabbit

• Chores
  • Streamlined continuous integration workflow for code coverage testing and analysis
  • Enhanced automated code quality reporting integration to improve quality tracking processes

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The CI workflow has been refactored to decouple test execution from coverage reporting. The test-with-coverage workflow now generates and uploads a coverage artifact, while a new upload-coverage workflow downloads that artifact and uploads it to Codecov as a separate step. This creates a two-stage pipeline triggered by pull requests on master.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflows ..github/workflows/test-with-coverage.yml	Renamed workflow from ci-build to test-with-coverage; refactored trigger condition to pull_request on master; condensed multi-step job into single test job with checkout, Go setup, and coverage test steps; replaced inline code coverage upload with coverage artifact creation at ./build/_output/coverage/coverage.txt
GitHub Actions Workflows ..github/workflows/upload-coverage.yml	New workflow that triggers after test-with-coverage completion; downloads coverage artifact; uploads to Codecov via codecov/codecov-action@v5 with token authentication and fail-on-error flag

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Two workflows now dance instead of one,
Artifacts hop between steps, coverage tests run,
Codecov awaits at the finish line bright,
Split tasks keep the pipeline lean and light!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: splitting the test-with-coverage.yml workflow into two separate workflows (test-with-coverage.yml and upload-coverage.yml).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.46%. Comparing base (10ca562) to head (0d40df2).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #505   +/-   ##
=======================================
  Coverage   78.46%   78.46%           
=======================================
  Files          53       53           
  Lines        2210     2210           
=======================================
  Hits         1734     1734           
  Misses        413      413           
  Partials       63       63           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sonarqubecloud]

Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud