Add coffeescript bindings

mattmcmanus / atom-ember-snippets

Ember.js ES6 snippets for Atom

https://atom.io/packages/ember-snippets

MIT License

38 stars 22 forks source link

Add coffeescript bindings #5

Closed jgarth closed 9 years ago

jgarth commented 9 years ago

Hi @seifsallam,

Thanks for publishing your snippets. I took the liberty of duplicating all snippets in CoffeeScript. I've also adjusted the Readme, Changelog and package.json. Would you be willing to pull them into your package? Let me know if any additional changes are necessary.

jgarth commented 9 years ago

Whoa, I could swear I saw no pull requests for this and now I can see there are three already... my bad! Feel free to close & ignore in favor of #2 / #3 / #4

seifsallam commented 9 years ago

@jgarth This looks more ready. Can you just update the .property and .observer to use new syntax Ember.computed and Ember.observer https://github.com/emberjs/guides/pull/110

jgarth commented 9 years ago

Thanks for the hint, the new syntax seems much nicer in conjunction with CoffeeScript.

seifsallam commented 9 years ago

@jgarth I've updated the package. Thanks for your PR :+1:

jgarth commented 9 years ago

Cheers! :+1: