-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Typings (typescript declarations) would be helpful #41
Comments
I'm actually working on a project with TypeScript right now, so I'm learning more about |
Let's keep this open. I can't work on it right, but might take a stab at it. |
@demurgos want to help out with this? |
Yup, |
@demurgos awesome! I didn't realize you made turbo-gulp - have you thought of converting it to a custom registry instead of wrapping gulp? https://github.com/gulpjs/undertaker#custom-registries |
Yes, this is something I had in mind for some time. I created this library about 2-3 years ago and it has gone through a lot of change. I did a big clean-up during the summer and I feel that it now reached a reasonable state. I still need to document it more because for the moment it remains a personal tool. My goal is to have a declarative interface for Gulp that is less customizable but provides a complete set of tools out-of-the-box. |
Updates the devDependency `rx@4` to `rxjs@5`. Closes gulpjs#41
Updates the devDependency `rx@4` to `rxjs@5`. Closes gulpjs#41
- Document support for RxJS 4 and 5 - Accept `Stream` as a return value instead of `Readable` - Remove dummy body in stream tests. Closes gulpjs#41
- Document support for RxJS 4 and 5 - Accept `Stream` as a return value instead of `Readable` - Remove dummy body in stream tests. Closes gulpjs#41
- Document support for RxJS 4 and 5 - Accept `Stream` as a return value instead of `Readable` - Remove dummy body in stream tests. - Reword some comments. Closes gulpjs#41
Thanks. |
The argument types and expected returns could be expressed in a typescript declarations
.d.ts
file, which would enable editors like VSCode to provide some help with using this function.The big restriction, re not supporting synchronous functions, might be hard to define in typescript. Though I write in TypeScript, the API to this is complex enough that writing the definitions file for this project may be over my head. But the expressive goodness of this should bubble up to packages like Gulp where mistakes over sync/async are more common.
The text was updated successfully, but these errors were encountered: