Skip to content

correct gulp.task signature#1653

Merged
phated merged 2 commits intogulpjs:masterfrom
makoConstruct:patch-1
May 13, 2016
Merged

correct gulp.task signature#1653
phated merged 2 commits intogulpjs:masterfrom
makoConstruct:patch-1

Conversation

@makoConstruct
Copy link
Contributor

gulp.task(name [, deps, fn]) means that only gulp.task(name, deps, fn) and gulp.task(name) are allowed, which is apparently not the case and it's a bit confusing. The correct notation, gulp.task(name [, deps] [, fn]), also permits for gulp.task(name, deps), and gulp.task(name, fn).

`gulp.task(name [, deps, fn])` means that only `gulp.task(name, deps, fn)` and `gulp.task(name)` are allowed, which is apparently not the case and it's a bit confusing. The correct notation, `gulp.task(name [, deps] [, fn])`, also permits for `gulp.task(name, deps)`, and `gulp.task(name, fn)`.
@phated
Copy link
Member

phated commented May 12, 2016

@makoConstruct thanks for the update; however, you must change the Table of Contents link at the top (I believe) because the updated header changes the way GitHub sets the ID on it.

@phated
Copy link
Member

phated commented May 13, 2016

Thanks!

@phated phated merged commit 5944c1b into gulpjs:master May 13, 2016
@skyl
Copy link

skyl commented Oct 2, 2017

What about the little description option? gulp.task(name, descr, fn) is also possible? I'm actually looking to have a description and dependencies but I can't find the signature for that offhand. Doesn't seem to support gulp.task(name, deps, descr, fn)

@phated
Copy link
Member

phated commented Oct 2, 2017

@skyl your comment doesn't belong here but I think you are looking for https://medium.com/gulpjs/gulp-sips-custom-task-metadata-9a2dc80ac7b1

@gulpjs gulpjs locked and limited conversation to collaborators Oct 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants