-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v4] Unexpected IsADirectoryError when uploading coverage #1303
Comments
@louwers can you provide how you are running the Codecov Action? |
Here is the entire workflow: name: upload-coverage
on:
workflow_run:
workflows: [linux-ci]
types:
- completed
jobs:
upload-coverage:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: ./.github/actions/download-workflow-run-artifact
with:
artifact-name: coverage-report
expect-files: "_coverage_report.dat"
- name: Upload coverage report
if: '!cancelled()'
uses: codecov/codecov-action@v4
with:
override_commit: ${{ github.event.workflow_run.head_sha }}
override_pr: ${{ github.event.workflow_run.pull_requests[0].number }}
token: ${{ secrets.CODECOV_TOKEN }}
files: "_coverage_report.dat"
disable_search: true
fail_ci_if_error: true
verbose: true |
@louwers I made a change to the underlying CLI, would you be able to run it again and see if that fixed it? |
@thomasrockhu-codecov It is still broken: Could you re-open? |
Thank you! |
I'm still seeing this issue with the latest release of the action - when is a new version of the action with the fix going to be released? |
@martincostello it should have already been released. can you open a new issue with links to CI and logs? |
Ever since upgrading to v4 we are seeing this error, with coverage uploading failing.
This directory is a submodule that is not checked out. But it should not affect coverage uploading.
There is just one file to be uploaded, which it correctly identifies (I am using the
files
field withdisable_search
):The text was updated successfully, but these errors were encountered: