Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

New branch, convert to es6 syntax, fix typo and add some more snippet. #8

Merged
merged 6 commits into from
Feb 20, 2017
Merged

New branch, convert to es6 syntax, fix typo and add some more snippet. #8

merged 6 commits into from
Feb 20, 2017

Conversation

letientai299
Copy link
Contributor

@letientai299 letientai299 commented Sep 15, 2016 •
edited
Loading

Please refer to #3 for the discussion on ES6 syntax.

Some notes about changes in this pull request.

  • I'd like to provide the explanation for all the snippets, so that, new users will be easier to pickup what they need.
  • Beside the UltisSnip snippets, I've also modified Readme to notice the changes and the limitation of my fork. Please review and update if needed.

@wgjtyu
Copy link

wgjtyu commented Jan 28, 2017

+1

@alexanderjeurissen
Copy link
Contributor

@justinj any blockers on getting this merged ?

@justinj justinj merged commit 53e803d into justinj:master Feb 20, 2017
@justinj
Copy link
Owner

justinj commented Feb 20, 2017

No, sorry! Looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@letientai299 @wgjtyu @alexanderjeurissen @justinj