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

RFE: parseYAML #91

Closed
derrickburns opened this issue Sep 20, 2020 · 4 comments
Closed

RFE: parseYAML #91

derrickburns opened this issue Sep 20, 2020 · 4 comments

Comments

@derrickburns
Copy link
Contributor

Can you add a function to parse a YAML string?

@lihaoyi-databricks
Copy link
Contributor

lihaoyi-databricks commented Sep 21, 2020

@derrickburns could you open this as an issue on the http://github.com/google/jsonnet repository? Sjsonnet largely tries to follow google/jsonnet's behavior as closely as possible, and doesn't make any language or library innovations on its own. If google/jsonnet adds such a function, we'll do so as well, if not we won't. Better to keep the language/library-related discussions centralized over on their repo

@olorin37
Copy link

olorin37 commented Apr 12, 2021

Hello, in fact it exists almost ready PR 888 for introducing std.parseYaml in google/jsonnet, and possibly it can be accepted when problems will be solved. There is also issue 460 for that (I do not know why it is not visible as linked issue). There is also ready waiting for C++ implementation PR 339 for go-jsonnet.

I mention it because it will be probably merged very soon to the reference implementation for sjsonnet and also I would like to use this option in future.

@olorin37
Copy link

olorin37 commented Jul 5, 2021

Currently std.parseYaml is already merged to both reference implementations (C++: google/jsonnet#899 and Go: google/go-jsonnet#339 versions). It is already on their masters, but not released in any tag yet.

So, it looks like good time to start implementation in sjsonnet.

@lihaoyi-databricks
Copy link
Contributor

This finally landed in 0.4.6 #142

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

3 participants