mattmcmanus / atom-ember-snippets

Ember.js ES6 snippets for Atom
https://atom.io/packages/ember-snippets
MIT License
38 stars 22 forks source link

adds module import snippets #26

Closed jbailey4 closed 7 years ago

jbailey4 commented 7 years ago

Merges atom-ember-module-snippets as discussed in https://github.com/jbailey4/atom-ember-module-snippets/issues/3

@mattmcmanus I added all the snippets as I had them originally, let me know if we want to add/remove some code. The only real differences I see are:

Closes #24

jbailey4 commented 7 years ago

@mattmcmanus I did try nested folders, but it seems atom doesn't currently support nested snippets, see this issue -> https://github.com/atom/snippets/issues/28

I'm okay with the import- prefix. I'll go ahead and add that.

jbailey4 commented 7 years ago

@mattmcmanus Okay good to go, added the import- prefix (https://github.com/mattmcmanus/atom-ember-snippets/pull/26/commits/b8e128c1f6b3c4fe3f4ccac32abdedca773acbd0)

jbailey4 commented 7 years ago

@mattmcmanus ping.

once this is merged, I'll deprecate atom-ember-module-snippets and instruct users to use this package instead

mattmcmanus commented 7 years ago

Sorry @jbailey4! Time got away from me. I'll cut a release tonight.

mattmcmanus commented 7 years ago

I just shipped 2.1.0 Let me know what you think

jbailey4 commented 7 years ago

@mattmcmanus No problem. Updated and looks good 👍 thanks for getting this in