-
Notifications
You must be signed in to change notification settings - Fork 32
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
Support for gulp 4.0 #31
Comments
You are correct. I was waiting on gulp4 to officially land before doing anything but you can see someone else's first stab at support here: #19 That PR is kinda rough though, so feel free to create a newer, cleaner PR or expand on what he did. Ideally the gulp4 version will support all the same features gulp-help does today in |
Yeah that would be sweet I have been using gulp 4 for about 6 months and it has been very stable for a long time. It is also much easier to use then the old version. But if you think the help will be pre-bundled then I don't see a reason to work on this. Maybe PR a change directly to gulp4 itself with help in it I suppose. |
sorry, what I meant was I don't want to lose any functionality in this module. From all I've read about gulp4 it doesn't seem they plan to implement any help beyond You can try to create a PR against gulp4 directly however I think they're mantra of simplicity will keep a feature like that out. I still think this module will stick around for people that want a more robust help menu. I just created a Thanks! |
cool ty |
I created a new API doc. Take a look and let me know what you think: https://github.com/chmontgomery/gulp-help/blob/gulp4/docs/API.md Trying to figure out the best way to structure the API that will be intuitive and useful to people. |
made some further progress today. I now have descriptions displaying using gulp's built-in convention for supplying descriptions (see latest on Supporting this way does have the benefit of having |
Cool stuff: However, I did just notice this may be wrapped into gulp4 anyway and we might not need this lib anymore.
Notice --task-simple and --tasks Saw this here, gulpjs/gulp#1159 |
@nmccready yes and no. yes, Depending on how all that shakes out, this repo could still be relevant or morph into the gulp4 way. Too early to tell yet. |
@chmontgomery I just tried the |
Any news with this one?! |
Just merged #47 . all examples are working now on the gulp4 branch |
So waiting for the release now. |
@lipis I don't have any plans for a "release". gulp 4 support only exists on a branch here in git. What's published to npm is the master branch, i.e. the version that works for gulp 3. You can do the following to reference gulp-help for gulp 4:
|
Github has rate limiting, so having this only installable via github causes issues for busy CICD pipelines. We've definitely been bit by having non-NPM dependencies in the past. Might you reconsider publishing this as a new major version or with a new name? |
Now that gulp installs the 4.x version by default now, it would make sense to have a new major release with the default support being for gulp 4. |
Can we make a new release to support Gulp 4? |
Looks like help bombs out on gulp4. Anyplans to support this? I am pretty familiar with 4.0 and can help out.
The text was updated successfully, but these errors were encountered: