Replace PyYAML by ruamel.yaml about pykwalify HOT 7 CLOSED

@Roysten This branch implmenets this behaviour https://github.com/Grokzen/pykwalify/tree/feature/ruamel.yaml, Look it through.

from pykwalify.

Do you have the history behind ruaemel.yaml, reasons for forking etc? Has there been any effort to merge back features into pyyaml?

from pykwalify.

I think this thread sums it up pretty well: https://bitbucket.org/xi/pyyaml/issues/59/has-this-project-been-abandoned. The original author of PyYAML does not actively maintain the project anymore, doesn't respond to emails etc., which makes it impossible for others to contribute.

from pykwalify.

@Roysten I will Not just drop pyyaml to replace it with anything else. But what i can do is to add a import test/check if ruamel is installed as a dependency and use that over pyyaml but allways default back to pyyaml becuase pyyaml is still so common in the wild and i have never seen a project that has ruamel.yaml as either supported or default implementation for loading yaml files.

from pykwalify.

@Grokzen Wow, I was not expecting it implemented this quicky! Looks great!

from pykwalify.

@Roysten The change was very small and easy to implement :D
It will however not be the default yaml lib for a while tho but people will be aware and maybe prepare or do the transition before 1.7.0 is out.

from pykwalify.

@Roysten Feature will be part of next release.

from pykwalify.