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
fix: Migrate away from deprecated pulp API #2610
Conversation
Once this is merged, the bioconda recipe needs to include the new version constraints |
Thanks! |
Fixing old Snakemake releases on bioconda here: bioconda/bioconda-recipes#45203 |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
by Handling backward compatibility within the code ref snakemake#2610 snakemake#2612
🤖 I have created a release *beep* *boop* --- ## [8.1.2](v8.1.1...v8.1.2) (2024-01-12) ### Bug Fixes * local mtime handling in case of storage plugins and cleaner error message for parallel storage retrieval ([#2611](#2611)) ([880b264](880b264)) * Migrate away from deprecated pulp API ([#2610](#2610)) ([fb26640](fb26640)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
I think the bioconda dependencies weren't adjusted 🙃 https://github.com/bioconda/bioconda-recipes/pull/45226/files |
@@ -46,7 +46,7 @@ install_requires = | |||
nbformat | |||
packaging | |||
psutil | |||
pulp >=2.0 | |||
pulp >=2.3.1,<2.9 # pulp introduced a breaking change from 2.7 to 2.8. Hence we should always pin to the minor version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mirrored to bioconda here: bioconda/bioconda-recipes#45234
Unfortunately not already for 8.1.2, just for 8.1.3
Description
The latest update of pulp 2.8.0 removed deprecated API calls still used by snakemake.
This PR updates the calls to their modern equivalents, as well as constrains the pulp version to the moment these got introduced (v2.3.1).
Fixes the issues described in #2607 and #2606.
QC
docs/
) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).