Bump codecov/codecov-action from 6 to 7#72
Conversation
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 6 to 7. - [Release notes](https://github.com/codecov/codecov-action/releases) - [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md) - [Commits](codecov/codecov-action@v6...v7) --- updated-dependencies: - dependency-name: codecov/codecov-action dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
dependencies
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| uses: codecov/codecov-action@v6 | ||
| uses: codecov/codecov-action@v7 | ||
| with: | ||
| file: TestResults/CoverageReport/Summary.xml |
There was a problem hiding this comment.
🚩 file parameter may be ignored by codecov-action v7
The codecov step at line 215 uses file: TestResults/CoverageReport/Summary.xml. In codecov-action v4+, the canonical input was renamed to files (plural). GitHub Actions silently ignores unrecognized inputs, so if file is not a recognized input in v7, the specified path would be silently ignored and codecov would fall back to auto-discovery. This was likely already the case with v6 as well (pre-existing), but worth verifying after the upgrade. The fail_ci_if_error: false at line 218 ensures CI won't break either way.
Was this helpful? React with 👍 or 👎 to provide feedback.
| run: dotnet tool restore | ||
|
|
||
| - name: 📊 Upload coverage reports to Codecov | ||
| uses: codecov/codecov-action@v6 | ||
| uses: codecov/codecov-action@v7 | ||
| with: | ||
| file: TestResults/CoverageReport/Summary.xml | ||
| flags: unittests |
There was a problem hiding this comment.
🚩 Coverage report file is never generated in the coverage-reports job
The coverage-reports job (lines 188-248) references TestResults/CoverageReport/Summary.xml for Codecov upload, but this job never runs any tests or generates any coverage report. The tests that collect coverage run in the build-metar job, and those artifacts are not downloaded in the coverage-reports job. The ReportGenerator tool is installed (line 207) but never invoked. This means the codecov upload step will always fail to find the file (falling back to auto-discovery, which also won't find anything since no tests ran). This is a pre-existing issue not introduced by this PR, but it means the codecov step is effectively a no-op regardless of the version bump.
(Refers to lines 187-218)
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Bumps codecov/codecov-action from 6 to 7.
Release notes
Sourced from codecov/codecov-action's releases.
Changelog
Sourced from codecov/codecov-action's changelog.
... (truncated)
Commits
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)