search

linkml / prefixmaps

Semantic prefix map registry
https://linkml.io/prefixmaps/
Apache License 2.0
10 stars 3 forks source link

Adjust `pyyaml` version #9

Closed caufieldjh closed 1 year ago

caufieldjh commented 1 year ago

Versions of pyyaml above 6.0 create a dependency conflict with koza. It appears to be safe to pin pyyaml to a compatible range here to fix the compatibility issue.

caufieldjh commented 1 year ago

Yes - all important considerations! To be honest I am also opening a competing PR for koza, just hadn't finished running the tests yet

caufieldjh commented 1 year ago

See https://github.com/monarch-initiative/koza/pull/96

pkalita-lbl commented 1 year ago

As far as I can tell the biggest incompatibility introduced by pyyaml@6.0 is dropping support for Python 2.7. I would hope that wouldn't cause an issue for koza.

caufieldjh commented 1 year ago

Sounds good - relaxed minimum pyyaml to 5.3.1 and all tests still pass.