[cleanup] ember-handlebars main.js Convert to single-quotes

[knownasilya]

No description provided.

[mmun]

~/Projects/ember.js/packages $ grep -r "from \"" . | wc -l
    2146
~/Projects/ember.js/packages $ grep -r "from \'" . | wc -l
     635

Looks like double quotes is more common. We should add a style guide.

[rwjblue]

Need to hold off on this until after #9827.

[knownasilya]

@mmun we should totally do single-quotes, they are less cluttering, which is pretty important for a big project like ember. Also easier to type.

@rwjblue did you ever get jscs working? We can update that and jshint, and work on making the code cleaner.

[mmun]

@knownasilya To each their own. I try to use singe quoted strings like I would use symbols in Ruby, and double quoted to mean content. But I wouldn't put up a fight if it was part of a style guide.

[rwjblue]

Either way using mixed quotes in the same file for the same thing is nuts (I generally follow the "single in JS and double in HTML/Handlebars" rule).

[knownasilya]

To each their own

That's exactly the problem 😉, and why a style guide is so necessary.

But I wouldn't put up a fight if it was part of a style guide.

👍

[knownasilya]

@rwjblue I've been following basically that pattern, although anything in {{}} uses single-quote, but lately I've been moving away from that due to htmlbars landing, making hbs and html more closely tied together.

[knownasilya]

#9849 👈 started an issue