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 licences command #3012

Merged
merged 4 commits into from
Jun 26, 2024

Conversation

mirpedrol
Copy link
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
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link

codecov bot commented May 31, 2024

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
34 checks passed
@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