Skip to content

Fix codecov-action input: file -> files [MOD-15112]#953

Merged
dor-forer merged 1 commit intomainfrom
dorer-mod-15112-codecov-files-input
May 5, 2026
Merged

Fix codecov-action input: file -> files [MOD-15112]#953
dor-forer merged 1 commit intomainfrom
dorer-mod-15112-codecov-files-input

Conversation

@dor-forer
Copy link
Copy Markdown
Collaborator

@dor-forer dor-forer commented May 5, 2026

Describe the changes in the pull request

Follow-up to #947. Renames the file input of codecov/codecov-action to files.

codecov/codecov-action removed the file input in v5.0.0 (renamed to files); v6 (the version we just bumped to) did not reintroduce it - see v6/action.yml. GitHub Actions silently drops unknown with: keys, so the previous file: ./bin/Linux-x86_64-debug/cov.info was being ignored. Combined with disable_search: true, the action would run successfully and upload no coverage report.

Which issues this PR fixes

  1. MOD-15112

Main objects this PR modified

  1. .github/workflows/coverage.yml

Mark if applicable

  • This PR introduces API changes
  • This PR introduces serialization changes

Pull Request opened by Augment Code with guidance from the PR author


Note

Low Risk
Low risk workflow-only change that corrects a misnamed Codecov action input so coverage reports are actually uploaded.

Overview
Fixes the Codecov upload step in .github/workflows/coverage.yml by renaming the codecov/codecov-action@v6 input from file to files, ensuring cov.info is actually discovered and uploaded (instead of being silently ignored).

Reviewed by Cursor Bugbot for commit 63cb890. Bugbot is set up for automated code reviews on this repo. Configure here.

Renaming codecov/codecov-action's 'file' input to 'files'. The 'file'
input was removed in v5.0.0 (renamed to 'files') and v6 did not
reintroduce it. Combined with 'disable_search: true', the old
'file:' key was silently ignored and no coverage report was uploaded.
@jit-ci
Copy link
Copy Markdown

jit-ci Bot commented May 5, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@dor-forer dor-forer marked this pull request as ready for review May 5, 2026 07:14
@dor-forer dor-forer enabled auto-merge May 5, 2026 07:21
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.92%. Comparing base (86cd1f5) to head (63cb890).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #953      +/-   ##
==========================================
- Coverage   96.92%   96.92%   -0.01%     
==========================================
  Files         130      130              
  Lines        7749     7739      -10     
==========================================
- Hits         7511     7501      -10     
  Misses        238      238              

☔ 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.

@dor-forer dor-forer added this pull request to the merge queue May 5, 2026
@dor-forer dor-forer removed this pull request from the merge queue due to a manual request May 5, 2026
@dor-forer dor-forer added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit 5ec2f77 May 5, 2026
25 of 32 checks passed
@dor-forer dor-forer deleted the dorer-mod-15112-codecov-files-input branch May 5, 2026 11:15
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.

3 participants