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

nf-test component for 'Nextflow for Genomics' #536

Merged
merged 18 commits into from
Mar 7, 2025

Conversation

pinin4fjords
Copy link
Collaborator

@pinin4fjords pinin4fjords commented Mar 4, 2025

To be merged after #534 (and re-pointed to master as base):

This is a port of (most of) the content formerly known as 'Hello nf-test'. Changes:

  • Stripped content now covered in the nf-test side-quest
  • Removed unnecessary nf-core references (e.g. the use of /local in module paths)
  • Added more specific commentary for genomics processes
  • Be more opinionated on setup usage.
  • Use actual workflow outputs to generate test data

Copy link

netlify bot commented Mar 4, 2025

Deploy Preview for nextflow-training ready!

Name Link
🔨 Latest commit 96f874e
🔍 Latest deploy log https://app.netlify.com/sites/nextflow-training/deploys/67c84b1a5637110008b70274
😎 Deploy Preview https://deploy-preview-536--nextflow-training.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pinin4fjords pinin4fjords changed the base branch from master to nf_test_genomics March 4, 2025 12:36
adamrtalbot
adamrtalbot previously approved these changes Mar 5, 2025
Copy link
Collaborator

@adamrtalbot adamrtalbot left a comment

Choose a reason for hiding this comment

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

Nice, I think this module needs some refinement in the context of the new module but it's good enough for now

Base automatically changed from nf_test_genomics to master March 5, 2025 12:47
@pinin4fjords pinin4fjords dismissed adamrtalbot’s stale review March 5, 2025 12:47

The base branch was changed.

@pinin4fjords pinin4fjords merged commit 09b226c into master Mar 7, 2025
8 checks passed
@pinin4fjords pinin4fjords deleted the nf_test_genomics_test branch March 7, 2025 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants