Both PyYAML and oyaml in requirements

[vreuter]

oyaml is billed as a drop-in replacement for PyYAML, but both are listed in the requirements. Would just one suffice? If both are needed, could a note be added to the requirements declaration about why that is?

[donaldcampbelljr]

Excellent. Thanks for the catch. You are correct; it was redundant. I've made the changes to Dev.

[nsheff]

is oyaml still needed or does pyyaml preserve ordering now that dicts are ordered by default?

• oyaml is a single-file library with only 53 lines of code
• oyaml is a drop-in replacement for PyYAML, which will load maps into collections.OrderedDict instead of regular dicts

See here:

https://stackoverflow.com/questions/13297744/pyyaml-control-ordering-of-items-called-by-yaml-load

But OrderedDict should no longer be necessary.

[nsheff]

In Python 3.7, the items-ordered feature of dict objects was declared an official part of the Python language specification. So, from that point on, developers could rely on dict when they needed a dictionary that keeps its items ordered.

https://realpython.com/python-ordereddict/

As long as we're not supporting python 3.6...

[nsheff]

It looks like we're still relying on oyaml, so this is not solved.

[donaldcampbelljr]

Ok, this should now be completed. I removed oyaml in favor of pyyaml per the above reading.