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.

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

YAML support #231

Closed
zekth opened this issue Mar 4, 2019 · 7 comments
Closed

YAML support #231

zekth opened this issue Mar 4, 2019 · 7 comments

Comments

@zekth
Copy link
Contributor

zekth commented Mar 4, 2019

As yaml is more and more used everywhere i think it's a nice feature to have in the deno_std.

We can port js-yaml or yaml-js for example.

Your thoughts?

@hayd
Copy link
Contributor

hayd commented Mar 4, 2019

I think toml is more popular in the rust community.

Personally I find yaml very brittle...

@j-f1
Copy link

j-f1 commented Mar 4, 2019

I’d suggest making a TOML parser instead, since YAML is extremely complex. For reference, a quick print-to-PDF yielded an 82-page-long spec for YAML and an 11-page-long spec for TOML (A4 size).

@zekth
Copy link
Contributor Author

zekth commented Mar 4, 2019

I don't like Yaml neither but i have to often use it that's why i'm asking for this. TOML is good too.

If everyone is ok for TOML let's close this one and open one for TOML.

@kevinkassimo what do you think?

@ry
Copy link
Member

ry commented Mar 4, 2019

I'm happy to have both in deno_std

@zekth zekth mentioned this issue Mar 4, 2019
@kevinkassimo
Copy link
Contributor

@zekth Why not both? ;)

@keroxp
Copy link
Contributor

keroxp commented May 13, 2019

@zekth
Copy link
Contributor Author

zekth commented May 13, 2019

FYI: https://github.com/nekobato/deno-xml-parser

Nice, would be nice to consider adding it to std. Also having the .toString() would be great.

@iugo iugo mentioned this issue Aug 2, 2022
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

No branches or pull requests

7 participants