Skip to content

Replace manual datadog-ci install with coverage-upload-github-action#490

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits into
DataDog:masterfrom
CarlosNietoP:carlos.nietopetinal/replace-coverage-upload-with-action
Mar 31, 2026
Merged

Replace manual datadog-ci install with coverage-upload-github-action#490
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits into
DataDog:masterfrom
CarlosNietoP:carlos.nietopetinal/replace-coverage-upload-with-action

Conversation

@CarlosNietoP
Copy link
Copy Markdown
Contributor

Summary

  • Replace the manual Node.js setup + npm install @datadog/datadog-ci steps in the upload-to-datadog job with DataDog/coverage-upload-github-action@v1
  • The action installs the datadog-ci standalone binary internally (no Node.js dependency needed), simplifying the workflow
  • Functionally equivalent: same API key, same coverage file (coverage.xml)

Test plan

  • Verify the upload-to-datadog job runs successfully on a merge to master
  • Confirm coverage data appears in Datadog after the upload

🤖 Generated with Claude Code

CarlosNietoP and others added 2 commits March 31, 2026 12:21
Use DataDog/coverage-upload-github-action@v1 instead of manually
installing Node.js and datadog-ci npm package. The action handles
binary installation internally without requiring Node.js.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@CarlosNietoP CarlosNietoP marked this pull request as ready for review March 31, 2026 10:31
@CarlosNietoP CarlosNietoP requested a review from a team as a code owner March 31, 2026 10:31
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.92%. Comparing base (e248d88) to head (e5ea97d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #490   +/-   ##
=======================================
  Coverage   84.92%   84.92%           
=======================================
  Files         190      190           
  Lines        9203     9203           
  Branches     4143     4143           
=======================================
  Hits         7816     7816           
  Misses        509      509           
  Partials      878      878           
Flag Coverage Δ
waf_test 84.92% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants