Skip to content
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

update webworkify-webpack to 2.1.5 #2174

Merged
merged 1 commit into from
Mar 16, 2019
Merged

Conversation

tjenkinson
Copy link
Member

This PR will...

update webworkify-webpack to 2.1.5 which fixes the issue which is causing hls.js to crash when used with create-react-app.

Why is this Pull Request needed?

So that hls.js works with create-react-app.

Resolves issues:

fixes #2064

which includes fix for issue with create-react-app

fixes #2064
@johnBartos
Copy link
Collaborator

Need to make sure that this works with harmony modules. I believe that I tried 2.1.4 when upgrading Babel and the modules weren't working correctly.

@johnBartos johnBartos added this to the 0.13.0 milestone Mar 13, 2019
@tjenkinson
Copy link
Member Author

tjenkinson commented Mar 13, 2019 via email

@johnBartos
Copy link
Collaborator

Ah ok, didn't know about 2.1.4 being released. Seems like this is working fine

Copy link
Collaborator

@johnBartos johnBartos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No change to the build size 👍

@tjenkinson
Copy link
Member Author

the tests passed in the push travis build. The other build is failing but is also showing a different commit for some reason

@tjenkinson tjenkinson merged commit bfe9ef2 into master Mar 16, 2019
@tjenkinson tjenkinson deleted the update-webworkify-webpack branch March 16, 2019 12:15
@johnBartos johnBartos modified the milestones: 0.13.0, 0.12.5 Jul 15, 2019
@johnBartos johnBartos modified the milestones: 0.12.5, 0.13.0 Aug 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hls.js in react application is not working
2 participants