-
Notifications
You must be signed in to change notification settings - Fork 968
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
mjml-core miss babel-runtime dep #1016
Comments
Hi @natesilva Thanks for reporting this, looks like babel-runtime is missing on a lot of packages ! Nice catch 👍 You can "bypass" temporarily it by adding the |
hi @iRyusa I have added
|
Really weird error considering that |
I'm using |
Looks like a dep issue in your project, you're using Note that gulp-mjml isn't up to date with MJML4 |
Ok, thanks. I think I have to rollback to previous mjml-core version. Do You have any plans to make mjml-4 compatible with gulp-mjml? |
Here mjmlio/gulp-mjml#34 but we're focusing on MJML4 fixes before. |
fix #1016 add babel-runtime dep on packages that could be imported
Hi @natesilva this has been fixed in the new release mjml4.2-beta, which you can install with npm install mjml, you can check if it solves the problem |
Thank you. It works. |
The
mjml-core
module doesn’t work as a standalone module. It errors on import.Use case: The main
mjml
module brings in a ton of dependencies that we don’t want (includingmjml-cli
which brings in the somewhat problematicfsevents
viachokidar
). We just need themjml2html()
function so I hoped to get that frommjml-core
.Reproduction Steps:
MJML version:
4.0.0
Node: 8.9.4
NPM: 5.6.0
The text was updated successfully, but these errors were encountered: