Open jonasfj opened 1 year ago
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. Such that we know that the formatting works, and behaves reasonably.
Perhaps, it'd be a good idea to start with https://github.com/dart-lang/yaml_edit/issues/33 (I just filed it, hehe) But really, adding the functions above is probably not too hard. The hard part is making sure we have tests, so that we're reasonably confident that these things work.
Sure i will try to take a look
Let me take a try :))
@jonasfj I will add some tests if there are \n as mentioned in #33
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!