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-unist-stringify? #122

Open
cherue opened this issue Dec 3, 2018 · 3 comments
Open

yaml-unist-stringify? #122

cherue opened this issue Dec 3, 2018 · 3 comments
Labels
enhancement New feature or request

Comments

@cherue
Copy link

cherue commented Dec 3, 2018

Any plans to implement stringify? This would make it a full unified processor 😇

@ikatyang
Copy link
Member

ikatyang commented Dec 5, 2018

No plan at the moment, I'll see if I have time to implement it.

(I've implemented a YAML printer in Prettier, though it requires original text to have better formatting, which does not seem to suit your use case.)

@ikatyang ikatyang added the enhancement New feature or request label Dec 5, 2018
@cherue
Copy link
Author

cherue commented Dec 5, 2018

Would you be willing to review and merge a PR that implements a stringify function?

If yes is there anything I should be aware of before I start?

@ikatyang
Copy link
Member

ikatyang commented Dec 5, 2018

👍 PRs are always very welcomed. The only thing you need to be noticed is that we need to parse -> stringify -> parse to ensure the AST won't be changed (excluding the position information).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants