Limitation: structure made to only build 1 theme

[bm-services]

We made a modification to build the theme you want (set a variable in gulpfile.js), if you are intereted in, take a look to this fork : https://github.com/BMServices/email-templates-sdk

We commented the CSS injection for now

[Scritik]

Hi @bm-services,

Thank you for you contribution. This is an interesting feature.
I think that you shouldn't force the THEME variable. Try to set it as param so we can run multiple Gulp at the same time to work on multiple themes.

Regarding the CSS injection, it was a temporary solution until MJML introduce its own tag: https://mjml.io/documentation/#mjml-style
We will remove it asap.

Do not hesitate to make a PR when all your changes are clean ;)

[bm-services]

Hi @Scritik,
we finally built the new version with theme name as an argument and improve some parts of the script.

A PR has just been made.