-
Notifications
You must be signed in to change notification settings - Fork 614
Codecov report fails CI although the bug with coverage was fixed in #1324 #1335
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
Comments
Codecov website shows "Not enough commits found with current parameters" if "last 7 days" is selected Looks like Codecov report is not uploaded on push to master |
Manually uploaded coverage reports for master from workspace. Will revisit when new PR is posted, and commit is merged to master |
The issue was fixed. |
ToDo: Verify if commit from PR #1337 appears in https://codecov.io/gh/aws/aws-sdk-js-v3/branch/master after merging to master, and successful build. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread. |
Describe the bug
Codecov shows coverage reduced although the issue with coverage was fixed
SDK version number
master
To Reproduce (observed behavior)
Create a new PR to the repository and see Codecov reports fail the CI
Expected behavior
CodeCov reports are successful, if code doesn't reduce coverage
The text was updated successfully, but these errors were encountered: