Closed heikki closed 1 year ago
Slush should definitely have gulp 4 support! I haven't had the time looking at it yet though.
Your work looks promising! I would gladly accept a PR, if you would like to provide one :)
Great! Still investigating some things, PR after that.
@heikki it would be great if you could make a PR of this to the v2
branch, even though you're not finished yet?
Ok. I keep this on hold until gulp-cli release with gulp 4 support is out.
gulp 4 eventually got its v4.0.0 tag and was released for comments as gulp@next
on Jan 1st 2018. An official gulp@4.0.0 npm package will follow once the community got some time for testing.
I'm linking a related issue and PR around removing a deprecated dependency (gulp-util
):
Hello guys,
Why is this is not going forward? Is almost a year of this issue. What can we do to make this happen? I trying to use this project but I can't go for it without gulp 4 support.
I want to make this happen, but I need help! See #64 for the latest discussion...
How I can help? I'm not very experienced but maybe I can help.
I tried to use gulp 4 with slush 1.1.1, when I slush my app, I got the following error:
/Users/jeantimex/.nvm/versions/node/v9.10.1/lib/node_modules/slush/bin/slush.js:106
gulpInst.start.apply(gulpInst, toRun);
^
TypeError: Cannot read property 'apply' of undefined
at /Users/jeantimex/.nvm/versions/node/v9.10.1/lib/node_modules/slush/bin/slush.js:106:20
at process._tickCallback (internal/process/next_tick.js:112:11)
at Function.Module.runMain (module.js:692:11)
at startup (bootstrap_node.js:194:16)
at bootstrap_node.js:666:3
If I rollback to gulp 3, the error is gone, anyone knows how to get around this? Here is my slush app: https://github.com/jeantimex/slush-html5-app, thanks!
Hi, do you have any thoughts regarding gulp 3/4 support?
I have done some digging in my fork: https://github.com/heikki/slush/tree/4.0 A lot of duplication can be removed because cli logic has now been separated to gulp-cli.