CGATOxford / UMI-tools

Tools for handling Unique Molecular Identifiers in NGS data sets
MIT License
481 stars 190 forks source link

Changed yaml.load to yaml.safe_load #592

Closed IanSudbery closed 1 year ago

IanSudbery commented 1 year ago

Recent debugging and testing revealed that changes to pyyaml meant changes were needed to how the testing sepcification was loaded.

TomSmithCGAT commented 1 year ago

This is the same as https://github.com/CGATOxford/UMI-tools/commit/7d49256441abd06592c747f3da65de4d52554612, right?

Not even sure why the diff looks like it does above since the safe_load is already in the master branch 🤷‍♂️

TomSmithCGAT commented 1 year ago

PR here: https://github.com/CGATOxford/UMI-tools/pull/542

IanSudbery commented 1 year ago

weird