Conversation
|
@jonschlinkert with #148 applied tests are passing. |
I'm not at all sure about that lol. Thank you for doing this and for bringing it to my attention. I was initially under the impression that edit: and also, thank you for taking the time to do this, and apologies for making you jump through hoops. I promise we'll get this merged and published as soon as we settle on which library to use. |
|
Would in fact go with |
|
@jonschlinkert any chance of getting this merged? 😅 |
|
@jonschlinkert ping 😅 |
|
@jonschlinkert 😿 - quite a lot of ppl are using this library (1M downloads per week). Would be really good if this could be merged anytime soon. |
|
@jonschlinkert ping on this, let me know if anything prevents merging. |
It no longer is. It shows yaml at 102.6 kB and js-yaml at 39 kB. |
|
@sakulstra Thank you for your contributions. @jonschlinkert Understandably you may have had to make other priorities. Would you be so kind to make @sakulstra a maintainer of this project? This nice library needs to keep up with the time. I am surprised that there is no TypeScript support (#105, #107, #181 and others). |
|
@pedzed Thanks, but I have no interest in maintaining this project at this point. which covers all our needs. |
use yaml instead of js-yaml
as requested in #145
@jonschlinkert are you sure
yamlis smaller thanjs-yaml(for some reason bundlephobia is broken for yaml and vscode showing bigger package size, might be wrong due to shaking though)yamlseems to behave slightly different as you can see in the two failing tests.