-
Notifications
You must be signed in to change notification settings - Fork 190
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
Add ability to install a nf-core module into a private module library #3174
Comments
@ewels This is one of the issues. |
Will #3083 also fix this? |
I don't think so, we would have to adapt the command a bit to allow installing modules on a modules repo. |
If you want to write an nftest case for a hybrid subworkflow then yes.
Otherwise the subworkflow can't execute in the library repo.
…On Thu, Sep 19, 2024 at 05:39 Júlia Mir Pedrol ***@***.***> wrote:
Will #3083 <#3083> also fix this?
I don't think so, we would have to adapt the command a bit to allow
installing modules on a modules repo.
But once combinations of different module sources are allowed, will you
need to install an nf-core module into a non-nf-core repo?
—
Reply to this email directly, view it on GitHub
<#3174 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEFHAFLREFQ6QSH4VLDZE23ZXKLTJAVCNFSM6AAAAABOMKALJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRQGUYDKNRWHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
oh, that's something to take into account! 👀 🚨 |
If #3083 is accepted you could set it up so that your tests install the subworkflow and then run the tests. This would of course necessitate having a
As it stands this sort of affects |
Description of feature
As a maintainer of a non-nfcore modules library using nf-core tooling, I would like to be able to install nf-core modules and subworkflows into my non-nfcore modules library for use building subworkflows.
This is currently prevented by this check . It could be adjusted to check if the
org_name
isnf_core
before failing.The text was updated successfully, but these errors were encountered: