-
Notifications
You must be signed in to change notification settings - Fork 19
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
Awesome, Async support please! #8
Comments
thanks mate. yeah, i know. can do when i have some free time. priorities and other works you know. but tell me, are you asking this because you have performance issues? |
Yeah no worries mate, I might take a look at implementing this myself when I've got a minute. Good to see the project is still active. I'm planning on using this on a large internal web application which I develop (100k+ daily users) and whilst I haven't ran into any performance related issues yet I just wanted to see if it was still in the pipeline. |
i see, sweet as. PRs welcome. or i ll do it in few months. or can do it now for a little bounty ;) |
+1 – I was installing this package when I realized that everything is sync. In a production server where a lot of things happen simultaneously, it's unusable. |
yeah i know and agree - again, PRs welcome |
I created #13 but ci fails b/c node 0.10 is installed |
Hello guys, it's been ages. I've ported this cool package to TS, should be easier to implement this request now. PRs welcome ;) |
Great stuff, love cleaning up my all temp files of x age with a 1 liner :)
Async support would be great, but otherwise one happy user here.
The text was updated successfully, but these errors were encountered: