Um, conda is failing in CI. Tried saying the modul...
# team-maintainers
c
Um, conda is failing in CI. Tried saying the module does not support conda. What to do? https://github.com/nf-core/modules/pull/10925
👀 1
m
add to the skip list
c
How do I do this
m
but since when does this tool not support conda?
c
It should
but it's affected by this
pkg_resources
drama
👀 1
m
yeah I would check that before trying to kill conda
c
I tried updating the bioconda recipe, but that didn't help
m
what about
pkg_resources
drama?
c
Actually it did help, but now there is another issue, and the tool isn't maintained
m
c
Basically now pin setuptools<81
m
even with a PR by @Usman
c
😞
m
Who's up to take over?
@Edmund made an orphanage for such tools
c
Fork?
O
m
you want to join in on the fun?
🎉 1
c
Going to try update the bioconda recipe again with
--no-build-isolation
in the meantime, but would be good to get the package off pkg_resources
party this 1
l
I've encountered a lot this issue with other process. The simplest way is to pin in the environment the python and setuptools version like for biohansel:
Copy code
---
# yaml-language-server: $schema=<https://raw.githubusercontent.com/nf-core/modules/master/modules/environment-schema.json>
channels:
  - conda-forge
  - bioconda
dependencies:
  - bioconda::bio_hansel=2.6.1
  - conda-forge::numpy=1.20.1
  - conda-forge::pandas=1.2.3
  - conda-forge::python=3.9.2
  - conda-forge::setuptools=49.6
🫠 1
m
That is going to break SO MANY THINGS
That's why we want conda lock files, right @Jonathan Manning?
l
What does it break ?
u
For what’s its worth, I had to write a specialised tool for this purpose in Go: https://github.com/GallVp/fa-lint We should probably deprecate fastavalidator and add it to the Conda skip list as it is not being maintained by the developer.
❤️ 2
c
I did see this tool, looks great. I haven't approved/merged fastavalidator yet, perhaps you would like to write a small advert for fa-lint on the PR?
u
@Cameron Lloyd I have added a deprecation message and a conda CI exclusion: https://github.com/nf-core/modules/pull/11012 The original commit is also included.
🙏 2