-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update js2coffee #1
Comments
No prob, I would accept a pr as well |
Cool. Looking at the source I see you have js2coffee embedded as a lib instead of a node module. Any reason in particular, or should I do the PR simply moving it to node_modules? |
Yeah, I made some changes to the source and fixed some bugs since it wasn't being maintained at the time. I'm not sure where it is now, but I'm happy to to move it into node_modules if that's useful to you |
It's a more general matter, really. If it was in node_modules, it would be trivial to update and keep updated. But if your changes are vital to the operation of the task, and they have not been reflected upstream, then we have a problem. So these are my questions now:
thanks |
I just took a quick look at the project and it's definitely better to use node_modules now. There was zero activity on the project before, but it appears to be in really good shape now. |
super. Will give it a shot as soon as I can and hopefully won't break On Sat, Oct 19, 2013 at 5:14 AM, Jon Schlinkert notifications@github.comwrote:
Alexandros Marinos Founder / Director, Rulemotion Ltd tel:**** +44 1483 68 8333 Surrey Technology Centre, 40 Occam Road**** mob:**** +44 7534 212 892** Surrey Research Park**** twt:**** @alexandrosm https://twitter.com/#!/alexandrosm Guildford GU2 7YG, UK**** Rulemotion Ltd is a limited company registered in England and Wales. IMPORTANT: This message from Rulemotion Ltd is private and confidential. If |
👍 |
Move js2coffee to npm module, update to js2coffee 0.2.2, make options work. #1
Any chance you can upgrade to version 0.2.2 as it has some features I would love to have?
Thanks
The text was updated successfully, but these errors were encountered: