[PR #1316] [MERGED] feat: Add code coverage support in CI #2156

New issue

Closed

opened 2026-10-29 21:33:12 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-29 21:33:12 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/Mooncake/pull/1316
Author: @LiYiMing-lg
Created: 1/1/2026
Status: ✅ Merged
Merged: 1/4/2026
Merged by: @stmatengss

Base: main ← Head: fix/688-code-coverage

📝 Commits (2)

db38b81 feat: add code coverage support in CI - Integrate coverage reporting into CI pipeline - Generate coverage reports for PRs and main branch - Add coverage tool integration - Closes #688
a09ebef fix: complete coverage implementation with all AI review fixes - Add coverage compiler flags via environment variables (CXXFLAGS/CFLAGS/LDFLAGS=--coverage) - Remove unsupported ENABLE_COVERAGE CMake option - Fix Chinese comments (replace with English) - Create valid lcov-compliant dummy file format - Update Codecov action from v3 to v4 - Add coverage failure tracking via output variable - Fix lcov exclusion patterns (remove '/build/') - Add coverage status check step

📊 Changes

1 file changed (+71 additions, -4 deletions)

View changed files

📝 .github/workflows/ci.yml (+71 -4)

📄 Description

Description

This PR adds code coverage reporting to the CI pipeline as requested in #688.

Changes

Integrated coverage tools (gcov/lcov) into CI workflow
Added coverage report generation for PRs and main branch builds
Configured coverage threshold checks

Closes #688

Checklist

Code coverage integrated into CI
Reports generated for PRs and main branch
No breaking changes introduced

Notes

This is a clean implementation focusing only on #688 requirements.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/Mooncake/pull/1316 **Author:** [@LiYiMing-lg](https://github.com/LiYiMing-lg) **Created:** 1/1/2026 **Status:** ✅ Merged **Merged:** 1/4/2026 **Merged by:** [@stmatengss](https://github.com/stmatengss) **Base:** `main` ← **Head:** `fix/688-code-coverage` --- ### 📝 Commits (2) - [`db38b81`](https://github.com/kvcache-ai/Mooncake/commit/db38b8101825025d72253a6489a9261ed64abb5f) feat: add code coverage support in CI - Integrate coverage reporting into CI pipeline - Generate coverage reports for PRs and main branch - Add coverage tool integration - Closes #688 - [`a09ebef`](https://github.com/kvcache-ai/Mooncake/commit/a09ebef2b0df19d40ed9a9cb35dc8a6a1ef9a212) fix: complete coverage implementation with all AI review fixes - Add coverage compiler flags via environment variables (CXXFLAGS/CFLAGS/LDFLAGS=--coverage) - Remove unsupported ENABLE_COVERAGE CMake option - Fix Chinese comments (replace with English) - Create valid lcov-compliant dummy file format - Update Codecov action from v3 to v4 - Add coverage failure tracking via output variable - Fix lcov exclusion patterns (remove '*/build/*') - Add coverage status check step ### 📊 Changes **1 file changed** (+71 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+71 -4) </details> ### 📄 Description ## Description This PR adds code coverage reporting to the CI pipeline as requested in #688. ## Changes - Integrated coverage tools (gcov/lcov) into CI workflow - Added coverage report generation for PRs and main branch builds - Configured coverage threshold checks ## Related Issues Closes #688 ## Checklist - Code coverage integrated into CI - Reports generated for PRs and main branch - No breaking changes introduced ## Notes This is a clean implementation focusing only on #688 requirements. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-29 21:33:12 +00:00