Skip to content

Conversation

@chief10
Copy link

@chief10 chief10 commented Aug 27, 2015

Added some es6 snippets to compliment what is already here. The commands for them are the same with the exception of being prepended with an 'e'.

@justinj
Copy link
Owner

justinj commented Aug 27, 2015

Nice! I can't help but feel like it might be better to have a completely separate set of snippets for this, though? It doesn't seem like a super common use case to be using the old syntax and new syntax at the same time (but feel free to correct me here). There's a set in #2, but I'm not sure how I want to handle it, I think a separate repo is probably best so these still exist for ES5. If either of @mileszs or @kevinold (or you!) want to make a new repo for it I'll link to it, otherwise I can make a new one.

@chief10
Copy link
Author

chief10 commented Aug 27, 2015

You are probably right about that. I'll for sure start a new repo over the weekend if no one else wants to -- that will give me a chance to fix a couple of issues I see with what I added.

Good work on this though man, and Thanks!

@chief10 chief10 closed this Oct 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants