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

fix: Fix compilation with node v12 by updating resemblejs #1237

Merged
merged 1 commit into from
Oct 28, 2019

Conversation

jmargeta
Copy link
Contributor

This change updates the resemblejs package from 3.1.0 to 3.2.3 and fixes VTK.js build with node v12.
The updated files come directly from running npm install resemblejs@3.2.3 (npm 6.11.3, node v12.11.1).

See also:
rsmbl/Resemble.js#176

…lejs

This change updates the resemblejs package from 3.1.0 to 3.2.3. The updated version uses a version
of the canvas library that can compile with node v12.
@thewtex thewtex force-pushed the update-resemblejs-fix-for-node12 branch from 2d42eea to 609e4a7 Compare October 28, 2019 19:44
@thewtex
Copy link
Member

thewtex commented Oct 28, 2019

@jmargeta thank you for contributing! 👏 👍 🌮

I reformatted your commit message to follow commitizen formatting and change from fix to chore because I have another dependency update to add for a fix release.

@thewtex thewtex merged commit 676023a into Kitware:master Oct 28, 2019
@jourdain
Copy link
Collaborator

🎉 This PR is included in version 11.7.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jourdain jourdain added the released Automated label label Oct 28, 2019
@jmargeta
Copy link
Contributor Author

Fantastic, thank you @thewtex !

@jmargeta jmargeta deleted the update-resemblejs-fix-for-node12 branch October 29, 2019 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Automated label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants