Skip to content
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

Better error message when using remove gulp.watch functionality #1002

Closed
phated opened this issue Apr 3, 2015 · 4 comments
Closed

Better error message when using remove gulp.watch functionality #1002

phated opened this issue Apr 3, 2015 · 4 comments

Comments

@phated
Copy link
Member

phated commented Apr 3, 2015

In the 4.0 branch, we removed the ability to use strings as watch tasks and it causes a cryptic error message TypeError: Arguments to path.resolve must be strings as noted in #999

We need to make this change/error more apparent.

@phated phated added this to the gulp 4 milestone Apr 3, 2015
@ddprrt
Copy link
Contributor

ddprrt commented Apr 5, 2015

Suggestion:

Error: watching %insert-glob-here%: watch task has to be either gulp.parallel, gulp.series or a function

This would achieve the following:

  • Giving a hint as to which watcher we refer to
  • Pointing to the new API of gulp.series and gulp.parallel here, which might help people moving from 3.* to 4.0

@phated
Copy link
Member Author

phated commented Apr 5, 2015

👍 would you be able to submit a PR adding that to index.js?

@tkellen
Copy link

tkellen commented Apr 5, 2015

Error: watching %insert-glob-here%: watch task has to be a function (optionally generated by using gulp.parallel or gulp.series)

phated added a commit that referenced this issue Apr 15, 2015
checking for watch parameters, fixes #1002
@phated
Copy link
Member Author

phated commented Apr 15, 2015

Fixed in c98b07a - thanks @ddprrt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants