-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: Bundle analysis notify task #224
Conversation
f1eabca
to
c02c10f
Compare
Codecov ReportAttention:
Changes have been made to critical files, which contain lines commonly executed in production. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #224 +/- ##
==========================================
- Coverage 98.21% 98.19% -0.02%
==========================================
Files 397 400 +3
Lines 30176 30506 +330
==========================================
+ Hits 29637 29956 +319
- Misses 539 550 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes This change has been scanned for critical changes. Learn more |
Codecov ReportAttention: @@ Coverage Diff @@
## main #224 +/- ##
==========================================
- Coverage 98.24% 98.22% -0.02%
==========================================
Files 366 369 +3
Lines 29475 29806 +331
==========================================
+ Hits 28957 29277 +320
- Misses 518 529 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #224 +/- ##
==========================================
- Coverage 98.24% 98.22% -0.02%
==========================================
Files 366 369 +3
Lines 29475 29806 +331
==========================================
+ Hits 28957 29277 +320
- Misses 518 529 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov Report
@@ Coverage Diff @@
## main #224 +/- ##
==========================================
- Coverage 98.24% 98.22% -0.02%
==========================================
Files 366 369 +3
Lines 29475 29806 +331
==========================================
+ Hits 28957 29277 +320
- Misses 518 529 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
|
db8546a
to
0472879
Compare
0472879
to
ad8af4e
Compare
This is a separate task for now and results in a separate PR comment. Ideally this would be part of the existing notify task but there's some significant refactoring work that needs to be done first to support that.
This new task runs at the end of the bundle analysis processor chain and posts a PR comment with information regarding the bundle analysis comparison across base/head commits.
Depends on the migration in codecov/codecov-api#320