-
Notifications
You must be signed in to change notification settings - Fork 16
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
Proposed auxiliary method wrapAsCustomStyledYamlNode #26
Comments
Maybe, there needs to be some logic around depth... I suppose it would be nice with an argument that always goes into flow-mode for a depth higher than some threshold, a reasonable default might be 16 (at that point block-mode becomes pretty sketchy to read, I'm just guessing). |
I need to try this mr Jonas @jonasfj |
@seifibrahim32, please do give it a try. I think one of the important aspects is to have proper tests. Perhaps, it'd be a good idea to start with #33 (I just filed it, hehe) |
Sure i will try to take a look |
Let me take a try :)) |
This function is better since If we used the normal wrapAsYamlNode it wont handle spaces \n or either indentation for scalars. |
We need to handle some tests so it clears this method's cases. |
This probably needs some test cases, and I'm sure the code can be improved further!
The text was updated successfully, but these errors were encountered: