Skip to content

Remove nf-core licences command#3012

Merged
mirpedrol merged 4 commits into
nf-core:devfrom
mirpedrol:remove-licenses-command
Jun 26, 2024
Merged

Remove nf-core licences command#3012
mirpedrol merged 4 commits into
nf-core:devfrom
mirpedrol:remove-licenses-command

Conversation

@mirpedrol

Copy link
Copy Markdown
Member

The nf-core licences command was not functional for DLS2 pipelines. The best approach is to remove it and start from scratch if needed.
Related to #2983

@mirpedrol mirpedrol changed the title Remove nf-core licenses command Remove nf-core licences command May 31, 2024
Comment thread CHANGELOG.md Outdated
@codecov

codecov Bot commented May 31, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.61%. Comparing base (feb5685) to head (aed32a9).
Report is 37 commits behind head on dev.

Current head aed32a9 differs from pull request most recent head 46e8f1a

Please upload reports for the commit 46e8f1a to get more accurate results.

Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mirpedrol mirpedrol requested a review from mashehu May 31, 2024 08:44
Co-authored-by: Júlia Mir Pedrol <mirp.julia@gmail.com>
@mirpedrol mirpedrol merged commit 1e93853 into nf-core:dev Jun 26, 2024
@mirpedrol mirpedrol deleted the remove-licenses-command branch June 26, 2024 12:43
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.

3 participants