Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect PyYAML version for bioblend 0.9.0 - 0.16.0 #412

Closed
hehao98 opened this issue Nov 18, 2021 · 1 comment
Closed

Incorrect PyYAML version for bioblend 0.9.0 - 0.16.0 #412

hehao98 opened this issue Nov 18, 2021 · 1 comment

Comments

@hehao98
Copy link

hehao98 commented Nov 18, 2021

Hello,

PyYAML 6.0 recently introduce a breaking change in its load() function (yaml/pyyaml#576). BioBlend also uses the load() function in bioblend/cloudman/launch.py from version 0.9.0 - 1.16.0 while all versions do not specify an upper version bound for pyyaml. This means that users that recently install these BioBlend versions may also install the incompatible PyYAML 6.0 package.

Although the code has already been fixed, it has not been published in PyPI. Also, users using these old versions are still affected by this breaking change. Maybe it will be better to add an upper version bound for PyYAML in all these BioBlend releases?

@nsoranzo
Copy link
Member

This was fixed in commit 2b85ca9 , first released in v0.17.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants