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

Remove nf-core modules check #878

Merged
merged 29 commits into from
Mar 15, 2021
Merged

Remove nf-core modules check #878

merged 29 commits into from
Mar 15, 2021

Conversation

ewels
Copy link
Member

@ewels ewels commented Mar 15, 2021

@ewels ewels added the WIP Work in progress label Mar 15, 2021
@codecov
Copy link

codecov bot commented Mar 15, 2021

Codecov Report

Merging #878 (494477f) into dev (bedbe53) will decrease coverage by 0.04%.
The diff coverage is 61.46%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #878      +/-   ##
==========================================
- Coverage   71.13%   71.08%   -0.05%     
==========================================
  Files          33       33              
  Lines        3596     3604       +8     
==========================================
+ Hits         2558     2562       +4     
- Misses       1038     1042       +4     
Impacted Files Coverage Δ
nf_core/list.py 82.23% <ø> (-0.08%) ⬇️
nf_core/modules/test_yml_builder.py 16.98% <0.00%> (-0.11%) ⬇️
nf_core/__main__.py 61.53% <39.28%> (-1.09%) ⬇️
nf_core/modules/create.py 63.75% <73.33%> (+1.75%) ⬆️
nf_core/modules/pipeline_modules.py 86.15% <74.19%> (-2.55%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bedbe53...5e82962. Read the comment docs.

nf_core/modules/create.py Outdated Show resolved Hide resolved
@drpatelh
Copy link
Member

Do we remove the update command too?

@ewels ewels marked this pull request as ready for review March 15, 2021 14:09
@ewels ewels added command line tools Anything to do with the cli interfaces DSL2 and removed WIP Work in progress labels Mar 15, 2021
@ewels
Copy link
Member Author

ewels commented Mar 15, 2021

Made an awesome fuzzy-finder interface when installing modules which I kind of love:

Mar-15-2021 15-12-09

@ewels
Copy link
Member Author

ewels commented Mar 15, 2021

Do we remove the update command too?

Yup, it's gone from the command line interface. I commented it out and left the stub as we do intend to put that in at a later date (I completely deleted check as that's part of lint).

@drpatelh
Copy link
Member

As mentioned on Slack we need to change where these modules are installed locally:

'directory/modules/local/process' dir called <tool_subtool.nf>

Need to change this install path to directory/modules/local/ i.e. without the process - based on recent module restructuring in #viralrecon

@ewels
Copy link
Member Author

ewels commented Mar 15, 2021

Dropped process dir in 336de7d 👍🏻

ewels and others added 3 commits March 15, 2021 17:39
Copy link
Member

@drpatelh drpatelh left a comment

Choose a reason for hiding this comment

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

Pleasure as always Sir! 💥

@drpatelh drpatelh merged commit a688788 into nf-core:dev Mar 15, 2021
@ewels ewels deleted the modules-cleanup branch March 15, 2021 22:05
@ewels ewels mentioned this pull request Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
command line tools Anything to do with the cli interfaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants