Draft
Conversation
This comment has been minimized.
This comment has been minimized.
kakkoyun
approved these changes
Mar 12, 2026
github-merge-queue bot
pushed a commit
that referenced
this pull request
Mar 13, 2026
## Summary Adds `code-coverage.datadog.yml` to configure Datadog Code Coverage ignore rules. This is a prerequisite for the coverage upload PR (#799) — the Datadog coverage reducer discovers this config file from the default branch via GitDB, so it must be merged to `main` before coverage uploads will apply ignore rules correctly. ### Ignore rules Excludes the following from coverage reporting (matching the existing Codecov behavior): - `samples/` and `_samples/` — test fixture directories - `_tools/` — build tooling - `dot_ast.proxies.go` — generated proxy file Includes both repo-relative paths and glob patterns to handle the Go module path resolver's suffix-matching limitation (where `_samples` Go module maps to `samples/` directory on disk). ### Context Part of the Codecov → Datadog Code Coverage migration. See #799 for the full upload integration. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Mirror Codecov upload exactly: same per-flag matrix, same files, same flags. Simplify code-coverage.datadog.yml to match codecov.yml ignore paths directly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ManuelPalenzuelaDD
force-pushed
the
add-datadog-coverage
branch
from
fa3de83 to
eb84a0b
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #799 +/- ##
==========================================
+ Coverage 65.72% 69.62% +3.90%
==========================================
Files 113 116 +3
Lines 7926 6900 -1026
==========================================
- Hits 5209 4804 -405
+ Misses 2192 1545 -647
- Partials 525 551 +26
🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
We're migrating Datadog repositories from Codecov to Datadog Code Coverage for tracking test coverage. This PR is the first step: it adds a Datadog coverage upload alongside the existing Codecov upload so we can run both systems in parallel and verify parity before switching over.
Changes
DataDog/coverage-upload-github-action@v1upload steps to thevalidate.ymlworkflow, mirroring the per-flag matrix strategy used by Codecovcode-coverage.datadog.ymlwith ignore paths matchingcodecov.yml(samples/,_tools/, auto-generated proxy files)DD_CI_API_KEYsecret (already available in this repo)continue-on-error: trueso neither blocks CIWhy are we doing this?
As part of a company-wide effort, we're consolidating code coverage reporting into Datadog's own Code Coverage product. This gives us:
Validation
CI needs to run to produce coverage data. Previous comparison on commit
e708754:The 2.11% gap is likely due to Go coverprofile statement-block-to-line mapping differences between the two systems. Still under investigation.
Next steps (not in this PR)
Once this PR is merged and we've confirmed Datadog coverage is stable over several commits:
CODECOV_TOKENsecretcodecov.ymlcode-coverage.datadog.ymlNo action needed from reviewers beyond normal review
This is a low-risk, additive change. The new upload steps run independently of the existing CI pipeline and cannot cause test failures.