YML loader support

[maxinthestudio]

Hi ! I was needing a yaml loader, so I took the existing I18nJsonLoader loader and I changed TWO lines:

Added this:

const yaml = require('js-yaml');

And changed this:

const data = JSON.parse(await readFile(file, 'utf8'));

for this:

const data = yaml.load(await readFile(file, 'utf8'), { json: true });

So, I was thinking, perhaps, you can add a protected method in I18nJsonLoader class ... lets say ... getDataAsJson() ... where any of us can extend that class, load whatever we want, convert that to json, and let the existing class continue its magic... what do you think ?

Thanks in advance !

[toonvanstrijp]

@maxinthestudio I like the idea of changing 2 lines of code to add yaml support. But I think we could better create a loader called I18nYamlLoader. And then ship the library with this loader as well.

[maxinthestudio]

Of course ! The only problem is ... all the code is the same ! Hahaha ! The only difference are those two sad sad lines !

[maxinthestudio]

https://github.com/toonvanstrijp/nestjs-i18n/pull/311/files

[rubiin]

@toonvanstrijp looks great

[maxinthestudio]

As you can see, changes are minor. And you have TWO loaders now. !