-
Notifications
You must be signed in to change notification settings - Fork 1.4k
build: Add shaka namespace types to generated externs #9596
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
Conversation
|
shaka-player.experimental.debug.externs.js --> shaka-player.dash.externs.js --> |
|
Bundle Size Report for PR #9596
|
|
Incremental code coverage: No instrumented code was changed. |
TAhub
left a comment
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.
The original issue singles out the shakaNamespaceType typedef in test/test/util/loader.js as something this change should remove the need for.
Some quick searching suggests the same typedef is used in multiple other test files too, such as test/ui/vr_ui_integration.js and test/test/util/manifest_generator.js.
I don't know if we can mark that issue closed until that typedef is refactored out. If you view that as outside of the scope of this particular PR, you could change this to just reference the issue rather than close it.
|
@TAhub do you want work on that part? |
|
Sure, I could make a follow-up PR if you want. |
Please! |
Issue #6762