-
Notifications
You must be signed in to change notification settings - Fork 12
Add new transplant_report task and logic
#1190
Conversation
✅ Sentry found no issues in your recent changes ✅ |
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #1190 +/- ##
=======================================
Coverage 97.70% 97.70%
=======================================
Files 454 457 +3
Lines 37032 37089 +57
=======================================
+ Hits 36182 36238 +56
- Misses 850 851 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
📢 Thoughts on this report? Let us know! |
transplant_report task and logic
| from app import celery_app | ||
| from database.enums import CompareCommitError, CompareCommitState | ||
| from database.models import CompareCommit, CompareComponent, CompareFlag | ||
| from database.models.reports import ReportLevelTotals, RepositoryFlag |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, this must be something I forgot to commit in a different PR.
It removes a wrong type, as the totals: ReportLevelTotals down below is wrong.
65b9171 to
7afd145
Compare
This new task is responsible for copying over the coverage report from one commit to another commit.
The commits for this feature:
transplant_reporttask and logic #1190transplantendpoint codecov-api#1276transplant-reportcommand codecov-cli#677