nf-core / tools

Python package with helper tools for the nf-core community.
https://nf-co.re
MIT License
242 stars 191 forks source link

3257 - Add linkify version to fix bioconda dependencies #3260

Closed pmoris closed 3 weeks ago

pmoris commented 3 weeks ago

As described in issue: https://github.com/nf-core/tools/issues/3257

Also see PR against the bioconda recipe (as a temporary fix): https://github.com/bioconda/bioconda-recipes/pull/51762

The bioconda version of nf-core tools produced errors when running commands that launched the trogon TUI. The reason was the missing package linkify, which gets pulled in as a dependency of textual in the PyPi build, but not in the conda-forge recipe. See https://github.com/Textualize/textual/blob/22770300252deb28d266fe4ed4766d6e2a2f5dd2/pyproject.toml#L44, https://github.com/conda-forge/textual-feedstock/blob/main/recipe/meta.yaml and https://github.com/nf-core/tools/issues/3257.

Unless the conda-forge recipe for textual gets fixed, the bioconda recipe fix might get lost when a new version is built against the PyPi version of nf-core. So I propose that we also explicitly specify linkify in the requirements.txt file.

Before this is merged, could someone please verify if the version pinning makes sense? As you can see in the bioconda PR, coderabbitai suggested pinning a specific version, but I think it got it the wrong way around.

PR checklist

pmoris commented 3 weeks ago

As mentioned in the issue, I also opened a PR against the conda-forge recipe for textual: https://github.com/conda-forge/textual-feedstock/pull/150

If/when it gets added there, it would not strictly be necessary to add linkify-it-py to the nf-core requirements anymore. But for the time being this would fix problems with people installing it through bioconda at least.

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.51%. Comparing base (eb4c237) to head (3a80996). Report is 9 commits behind head on dev.

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

pmoris commented 3 weeks ago

The conda-forge recipe update already got merged, so hopefully any future releases/builds should work.

That means that this PR can be closed without merging it, but the bioconda-recipe one should still be merged to fix the current version of the package.