Skip to content

Compatibility with Node 12.0 - Upgrade to Gulp 4 #118

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

Merged
merged 1 commit into from
Jul 14, 2019
Merged

Compatibility with Node 12.0 - Upgrade to Gulp 4 #118

merged 1 commit into from
Jul 14, 2019

Conversation

mon-compte-github
Copy link
Contributor

@mon-compte-github mon-compte-github commented Jul 9, 2019

Hi,

I've started to work with you amazing html template months ago. I used it for an angular dashboard app. It's so easy to work with, I love it. Thank you very much for your work :-)

A few days ago, I had to fix stuff on my project but it didn't work anymore : my node installation had been updated to v12 and node-sass wasn't happy. So I decided to make my first pull request. Here it is ! Please be kind ;-)

I had to upgrade gulp to v4 so I corrected the building scripts, and add uglification to app and vendor js (should be optional but I dunno how to do it the right way with gulp).

Tell me if I made something wrong.
Best regards
Jay

@modularcoder
Copy link
Member

@mon-compte-github thanks a lot for the PR, will check this out!

@modularcoder modularcoder changed the base branch from master to v1-dev July 14, 2019 22:28
@modularcoder
Copy link
Member

modularcoder commented Jul 14, 2019

@mon-compte-github I'll merge the PR to the dev branch.
There are still some compilation errors related to node-gyp and fsevents, but seems like the can be fixed with the new lockfile.

Thanks a lot for the PR! I'll add some minor changes and make a new release

@modularcoder modularcoder merged commit 3672438 into modularcode:v1-dev Jul 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants