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

Dev -> Master for v2.9 #2349

Merged
merged 327 commits into from
Jun 30, 2023
Merged

Dev -> Master for v2.9 #2349

merged 327 commits into from
Jun 30, 2023

Conversation

mirpedrol
Copy link
Member

No description provided.

MatthiasZepper and others added 30 commits May 25, 2023 14:27
… the new ability to download multiple revisions at once. This resulted in loooooooong filenames.
…load. Ultimately, this now means that I also have to implement multiple version downloads for the classic download. Just downloading the first doesn't seem to make sense from a UX perspective.
…link from singularity to apptainer on the system.
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
README.md Outdated
@@ -1181,7 +1181,7 @@ The `nf-core subworkflows create` command will prompt you with the relevant ques

<!-- RICH-CODEX
working_dir: tmp
before_command: git clone https://github.com/nf-core/modules.git && cd modules
before_command: rm -rf ~/.config/nfcore/nf-core/modules && git clone https://github.com/nf-core/modules.git && cd modules
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
before_command: rm -rf ~/.config/nfcore/nf-core/modules && git clone https://github.com/nf-core/modules.git && cd modules
before_command: cd modules

We don't have to clone the repo again, it should already be there from the modules commands before (instead of cd we could remove the whole before_command and have working_dir: tmp/modules

@@ -3,6 +3,6 @@
Shouldn't do much, as everything is under subcommands.
"""

import pkg_resources
import importlib.metadata as importlib_metadata
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Admittedly, I am newly using pkg_resources in Downloads:

from pkg_resources import parse_version as VersionParser

It is just something I came by when searching for suggestions online how to reliably sort semantic versions correctly. I am open to change that, in case dropping pkg_resources is desired?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need to drop it, but can think about it if there's a better solution :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far, I am just aware that distutils is deprecated, which I had used initially before switching to pkg_resources. So is importlib the new thing to use (at least for tools)?

@mirpedrol mirpedrol merged commit e5ce6ce into master Jun 30, 2023
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

Successfully merging this pull request may close these issues.

10 participants