moviepilot/chaplin — Issue #5: Code style improvement
Author: paulmillr
Created: 2012-03-02 12:18:19+00:00
State: open

First of all, thanks guys, the project is neat, perhaps @brunch should integrate it as a part of alternate template.

But there are some differences with coffeescript code style [best practices](https://github.com/polarmobile/coffeescript-style-guide). I propose to:

* prohibit standalone `@` and use `this` instead
* remove spaces around objects `{ a: b }` → `{a: b}`
* do some newline fixes (prohibit two+ blank lines in classes / functions, move comments just under code blocks instead of 1-line spacing between them)
* shorten lines to 80 symbols
* rename `e` variables to `event` for clarity

Again, all of these (except the last one) had been approved by Jeremy. If you're interested, I'll be happy to send a pull request.