-
Notifications
You must be signed in to change notification settings - Fork 60
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
Create GCContentTrack type #4288
Conversation
@@ -49,11 +49,13 @@ const ErrorMessage = ({ | |||
onReset?: () => void | |||
}) => { | |||
const str = `${error}` | |||
const str2 = str.indexOf('expected an instance of') | |||
const str3 = str2 !== -1 ? str.slice(0, str2) : str |
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.
this change significantly shortens error messages
ad25fa8
to
ec0eff8
Compare
ec0eff8
to
8437a87
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4288 +/- ##
==========================================
+ Coverage 62.60% 62.64% +0.03%
==========================================
Files 1088 1092 +4
Lines 31496 31498 +2
Branches 7528 7527 -1
==========================================
+ Hits 19719 19732 +13
+ Misses 11602 11593 -9
+ Partials 175 173 -2 ☔ View full report in Codecov by Sentry. |
additional tooling could allow a user to auto-create this track from the 'add-assembly' cli or from gui potentially. but, might be ok for now |
this incorporates the "GC content display" but is a separate track type
it requires cloning the display type since a display can only be registered to one track type at a time