8000 purgedups/calcuts nftest migration by hyunhwan-jeong · Pull Request #8471 · nf-core/modules · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

purgedups/calcuts nftest migration #8471

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

purgedups/calcuts nftest migration #8471

wants to merge 6 commits into from

Conversation

hyunhwan-jeong
Copy link
Contributor
@hyunhwan-jeong hyunhwan-jeong commented May 14, 2025 •
edited by LouisLeNezet
Loading

PR checklist

Closes #7672

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the module conventions in the contribution docs
  • If necessary, include test data in your PR.
  • Remove all TODO statements.
  • Emit the versions.yml file.
  • Follow the naming conventions.
  • Follow the parameters requirements.
  • Follow the input/output options guidelines.
  • Add a resource label
  • Use BioConda and BioContainers if possible to fulfil software requirements.
  • Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
    • For modules:
      • nf-core modules test <MODULE> --profile docker
      • nf-core modules test <MODULE> --profile singularity
      • nf-core modules test <MODULE> --profile conda
    • For subworkflows:
      • nf-core subworkflows test <SUBWORKFLOW> --profile docker
      • nf-core subworkflows test <SUBWORKFLOW> --profile singularity
      • nf-core subworkflows test <SUBWORKFLOW> --profile conda

@hyunhwan-jeong hyunhwan-jeong requested review from nh13, mashehu and mirpedrol and removed request for nh13 May 14, 2025 20:16
@hyunhwan-jeong hyunhwan-jeong self-assigned this May 14, 2025
@nh13 nh13 removed their request for review May 16, 2025 01:05
@nh13
Copy link
Member
nh13 commented May 16, 2025

Sorry, I am not able to review at this time

@famosab
Copy link
Contributor
famosab commented May 20, 2025

See also #8473

famosab
famosab reviewed May 20, 2025
View reviewed changes
famosab
famosab requested changes May 20, 2025
View reviewed changes
@github-project-automation github-project-automation bot moved this from Todo to In Progress in nf-test Migration May 20, 2025
@famosab famosab changed the title purgedups/getseq nftest migration purgedups/calcuts nftest migration May 20, 2025
hyunhwan-jeong and others added 2 commits May 20, 2025 21:28
@famosab famosab mentioned this pull request May 21, 2025
Open
17 tasks
@famosab famosab linked an issue May 22, 2025 that may be closed by this pull request
nf-test migration: purgedups/calcuts #7671
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@famosab famosab famosab requested changes

@mashehu mashehu Awaiting requested review from mashehu

@mirpedrol mirpedrol Awaiting requested review from mirpedrol

Requested changes must be addressed to merge this pull request.

Assignees

@hyunhwan-jeong hyunhwan-jeong

Labels
nf-test
Projects
nf-test Migration
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

nf-test migration: purgedups/getseqs nf-test migration: purgedups/calcuts
4 participants
@hyunhwan-jeong @nh13 @famosab @hyunhwan-bcm
0