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

Use js-yaml over yaml-js #25

Closed
devinus opened this issue Sep 3, 2015 · 2 comments
Closed

Use js-yaml over yaml-js #25

devinus opened this issue Sep 3, 2015 · 2 comments
Assignees
@jxson
Labels
Feature
Milestone
v2.0

Comments

@devinus
Copy link

devinus commented Sep 3, 2015

https://github.com/nodeca/js-yaml is more maintained and faster.
@jxson
Copy link
Owner

jxson commented Sep 16, 2015

Thanks for the suggestion @devinus I will look into it swapping it out. Any idea of the test suite is as thorough?

This is kind of related to #22, it would be nice to be able to pass in your own parser as an option (even if it is just a different yaml parser).

@jxson jxson added the Feature label Sep 16, 2015
@jxson jxson added this to the v2.0 milestone Sep 16, 2015
@jxson jxson self-assigned this Sep 16, 2015
@notslang
Copy link
Contributor

👍

notslang added a commit to notslang/front-matter that referenced this issue Nov 28, 2015
@notslang
switch parser to js-yaml
4683c5d 
closes jxson#25
@jxson jxson closed this as completed in 9de8979 Dec 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jxson jxson

Labels
Feature
Projects
None yet
Milestone
v2.0
Development

No branches or pull requests
3 participants
@devinus @jxson @notslang