-
Notifications
You must be signed in to change notification settings - Fork 34
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
Use fiber for async code #307
Comments
@totty90 mind expanding on this? |
By using fibers your code can look like this
While with the current way of doing things is kind a pain. |
Hi @totty90 , thanks for the great idea. If you are interested in taking a stab at implementing this I would love to take a look and we can evaluate further. You are correct that using the pattern above would remove quite a bit of complexity when writing tests. Luckily we have a pretty extensive set of unit and integration tests for this library so you can quite easily make a pass on this and be able to see if it is going to be possible. Thanks again. |
You are welcome, I've just moved all my server side services to fibers and 100 lines of code with async just transformed in 10 readable lines. Is like magic. No problems to debug yet. |
ok @totty90 well if you are interested, we are very open to improvements. |
This is pretty much the same problem that I had to solve with integrating Meteor with pioneer.js You can see how I did it with space-pioneer, an acceptance testing framework for Meteor.js based on pioneer.js |
Here it is cucumber + fibers + more goodies: https://github.com/hackhat/cucumberry |
No description provided.
The text was updated successfully, but these errors were encountered: