-
Notifications
You must be signed in to change notification settings - Fork 56
Switch to more standardish/expected callback(error, data) pattern #15
Comments
+1 This issue and the mess the old Have you considered taking advantage of some tools and libraries to make development easier and the code simpler? CoffeeScript gives us an abstraction for classes, string interpolation, etc. and Streamline really simplifies async code. Let me know if you're interested in collaborating. |
Sorry, but I'd like to stick to straight JS and callbacks for node-twitter. It'd be nice if you renamed your repo so no one is confused about the relationship between node-twitter and your own library. Thanks. (The cookie issue is being resolved at the moment -- see the pull request.) |
I respect your decision. Regarding the name, no worries, once your library doesn't rely on Nice work on node-twitter :) |
I'll move it to a different module (included with node-twitter or separate, haven't decided). |
Cool, looking forward to check it out. |
+1 loving the library and this would be really nice to have |
Check out the request branch for a first cut of this. Every post-0.1.18 release will use the standard node callback pattern. :-) |
Sweet! Looking forward to it. Thanks @jdub. |
Just waiting on some fixes to request before merging the new request (and standard callback pattern) branch. |
Here Line 104 in 5fa1af0
Is this an oversight or did you decide not to use the P.S. Here I am, three years later, working for a different company and a different product and ran into the same issue. Made me chuckle seeing myself having commented on this back then 😉 /cc @aseemk |
No description provided.
The text was updated successfully, but these errors were encountered: