-
Notifications
You must be signed in to change notification settings - Fork 24
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: Grab the plugin name for the given bundle #3535
feat: Grab the plugin name for the given bundle #3535
Conversation
Bundle ReportChanges will increase total bundle size by 388 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
Bundle ReportChanges will increase total bundle size by 388 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
@@ Coverage Diff @@
## main #3535 +/- ##
=======================================
Coverage 98.90% 98.91%
=======================================
Files 806 806
Lines 14485 14484 -1
Branches 4116 4108 -8
=======================================
Hits 14327 14327
+ Misses 151 150 -1
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
@@ Coverage Diff @@
## main #3535 +/- ##
=======================================
Coverage 98.90% 98.91%
=======================================
Files 806 806
Lines 14485 14484 -1
Branches 4116 4115 -1
=======================================
Hits 14327 14327
+ Misses 151 150 -1
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #3535 +/- ##
=======================================
Coverage 98.90% 98.91%
=======================================
Files 806 806
Lines 14485 14484 -1
Branches 4116 4115 -1
=======================================
Hits 14327 14327
+ Misses 151 150 -1
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
bd44b3a
to
55a3612
Compare
55a3612
to
7caad44
Compare
@@ -54,21 +55,23 @@ export function useBundleAssetsTable({ | |||
} | |||
}, [overview?.oldestCommitAt, today, trend]) | |||
|
|||
return useBundleAssets({ |
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.
i like this change
7caad44
to
06e670d
Compare
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
@@ Coverage Diff @@
## main #3535 +/- ##
=======================================
Coverage 98.90% 98.91%
=======================================
Files 806 806
Lines 14485 14484 -1
Branches 4109 4108 -1
=======================================
Hits 14327 14327
+ Misses 151 150 -1
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Description
This PR updates the query options for grabbing a list of assets used to display the asset table, so that we can determine if we want to render the new path column in the asset table
Closes codecov/engineering-team#2957
Notable Changes
useBundleAssetsTable