-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat: bring yaml #250
feat: bring yaml #250
Conversation
```js | ||
bring fs; | ||
bring yaml; | ||
bring cloud; | ||
|
||
let file = fs.readFile("./test.yml"); | ||
let value = yaml.parseValue(file); | ||
yaml.stringifyValue(json); | ||
|
||
new cloud.Function(inflight () => { | ||
let value = yaml.parse(file); | ||
yaml.stringify(json); | ||
}); | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should totally include this in the built-in standard library!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we already have some yaml stuff in the standard library?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree this should be part of the stdlib.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apparently we already have fs.readYaml
.. was not aware of it
indent: num?; | ||
} | ||
|
||
pub class Util { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the benefit of this duplication? Why not just make the extern methods public?
No description provided.