Enable more verbose codecov reporting #791
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This re-enables comments, patch statuses, the verbose flag in uploads, and does an ls in the pytest workflow to verify coveage reports are generated properly.
I'll plan to leave this as is for a while until I'm convinced I understand how codecov works. I think the main problem is the stale PRs don't have base reports, and rebasing sometimes doesn't actually work to update the reporting.
Motivation and Context
PRs aren't showing their coverage status when reports are uploaded. This seems to mostly relate to the fact that the head commits don't have reports, so codecov can't report a difference in coverage. But I tried rebasing #778 to a commit that has a report and it still didn't work.
I'd like to not have codecov create comments, but having that on for now will be helpful, as those seem to report before the status checks.
How Has This Been Tested?
I've been testing a bunch of configuration changes and scenarios in a test repo. It's sort of convinced me that the configuration we have should be fine, but I still think these changes will be helpful until things are working smoothly.
Types of changes
Checklist: