-
Notifications
You must be signed in to change notification settings - Fork 220
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
Possible switch to CoffeeScript #111
Comments
It does seem a lot more readable. How much effort would it take to convert the entire BQ source? |
I don't think it will end up being that much work. The main issue is going to be getting it to work with RequireJS, but I think I've figured it out. |
k, start converting the rest then and see how you go. If you still feel it's a good idea after you've done a bunch more of the files, then lets bring it up on the mailing list again to see if someone has objections. It'll probably be ok though... 😄 |
I've now converted four files to CoffeeScript on my coffeescript branch. I still think that it's a good idea to convert the rest, so I'll send an email to the mailing list. |
For anyone else reading along, we're trying an experimental conversion of everything to CoffeeScript in the "coffeescript" branch here: https://github.com/browserquest/BrowserQuest/tree/coffeescript It's ongoing work, with the majority of the client side automatically converted using js2coffee. Feel free to assist if desired. 😃 |
Closing this, as it's probably better to track it in PR #124, where the work is happening. |
I've converted a couple files to CoffeeScript at Aaron1011@618b869. CoffeeScript is a language which compiles into JavaScript, trying to make it easier to write.
There has been some discussion on the mailing list earlier about switching to CoffeeScript. I've converted these files so that people can take a look at it, and decide if switching would be a good choice. You might also want to check out the CoffeeScript website, which goes into much further detail about the language.
The text was updated successfully, but these errors were encountered: