Skip to content

Conversation

@vdauwera
Copy link
Collaborator

@vdauwera vdauwera commented Nov 25, 2025

Toying with some ideas to improve the layout of the Side Quests.

  • Distinguish and highlight Learning goals and Prerequisites with subheadings in the intro part
  • Rename the warmup section (Get started) and give it more structure
    • Open the training codespace
    • Move into the project directory
    • Explore the materials
    • Scenario
    • Readiness checklist
  • Summary: associate the code examples with the "what you've learned" recaps and streamline the AI-generated text. Some of it is repetitive.
  • Add/rework "What's next?" sections

@vdauwera vdauwera self-assigned this Nov 25, 2025
@netlify
Copy link

netlify bot commented Nov 25, 2025

Deploy Preview for nextflow-training ready!

Name Link
🔨 Latest commit cb60d65
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-training/deploys/692f4f1c901406000892bb5c
😎 Deploy Preview https://deploy-preview-727--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 project configuration.

@vdauwera vdauwera marked this pull request as ready for review December 2, 2025 19:08
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

Nextflow linting complete!

❌ 5 files had 33 errors
✅ 143 files had no errors

💡 Tip: Click filename locations to go directly to that code.

View all 33 issues
Type Location Message
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:11:1 Invalid include source: '/home/runner/work/training/training/hello-nextflow/solutions/5-hello-containers/modules/sayHello.nf'
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:12:1 Invalid include source: '/home/runner/work/training/training/hello-nextflow/solutions/5-hello-containers/modules/convertToUpper.nf'
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:13:1 Invalid include source: '/home/runner/work/training/training/hello-nextflow/solutions/5-hello-containers/modules/collectGreetings.nf'
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:24:5 sayHello is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:27:5 convertToUpper is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:27:20 sayHello is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:30:5 collectGreetings is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:30:22 convertToUpper is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:33:5 collectGreetings is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:36:11 collectGreetings is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:11:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/sayHello.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:12:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/convertToUpper.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:13:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/collectGreetings.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:14:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/cowpy.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:24:5 sayHello is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:27:5 convertToUpper is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:27:20 sayHello is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:30:5 collectGreetings is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:30:22 convertToUpper is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:33:5 cowpy is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:33:11 collectGreetings is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:36:20 cowpy is not defined
Error hello-nf-core/solutions/core-hello-part2/nextflow.config:154:17 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/core-hello-part2/conf/test_full.config'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:1:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/validate_name.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:2:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/say_hello.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:3:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/timestamp_greeting.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:11:24 VALIDATE_NAME is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:12:24 SAY_HELLO is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:13:26 TIMESTAMP_GREETING is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:1:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/say_hello_upper.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:2:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/reverse_text.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:10:20 SAY_HELLO_UPPER is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:11:23 REVERSE_TEXT is not defined

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

Nextflow linting complete!

❌ 5 files had 33 errors
✅ 143 files had no errors

💡 Tip: Click filename locations to go directly to that code.

View all 33 issues
Type Location Message
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:11:1 Invalid include source: '/home/runner/work/training/training/hello-nextflow/solutions/5-hello-containers/modules/sayHello.nf'
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:12:1 Invalid include source: '/home/runner/work/training/training/hello-nextflow/solutions/5-hello-containers/modules/convertToUpper.nf'
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:13:1 Invalid include source: '/home/runner/work/training/training/hello-nextflow/solutions/5-hello-containers/modules/collectGreetings.nf'
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:24:5 sayHello is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:27:5 convertToUpper is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:27:20 sayHello is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:30:5 collectGreetings is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:30:22 convertToUpper is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:33:5 collectGreetings is not defined
Error hello-nextflow/solutions/5-hello-containers/hello-containers-2.nf:36:11 collectGreetings is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:11:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/sayHello.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:12:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/convertToUpper.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:13:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/collectGreetings.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:14:1 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/composable-hello/modules/cowpy.nf'
Error hello-nf-core/solutions/composable-hello/hello.nf:24:5 sayHello is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:27:5 convertToUpper is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:27:20 sayHello is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:30:5 collectGreetings is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:30:22 convertToUpper is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:33:5 cowpy is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:33:11 collectGreetings is not defined
Error hello-nf-core/solutions/composable-hello/hello.nf:36:20 cowpy is not defined
Error hello-nf-core/solutions/core-hello-part2/nextflow.config:154:17 Invalid include source: '/home/runner/work/training/training/hello-nf-core/solutions/core-hello-part2/conf/test_full.config'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:1:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/validate_name.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:2:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/say_hello.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:3:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/timestamp_greeting.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:11:24 VALIDATE_NAME is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:12:24 SAY_HELLO is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/greeting.nf:13:26 TIMESTAMP_GREETING is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:1:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/say_hello_upper.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:2:1 Invalid include source: '/home/runner/work/training/training/side-quests/solutions/workflows_of_workflows/modules/reverse_text.nf'
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:10:20 SAY_HELLO_UPPER is not defined
Error side-quests/solutions/workflows_of_workflows/workflows/transform.nf:11:23 REVERSE_TEXT is not defined

Copy link
Collaborator

@pinin4fjords pinin4fjords left a comment

Choose a reason for hiding this comment

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

Nice work!

The standardized structure for intro/outro sections is a clear improvement - having consistent "Learning goals", "Prerequisites", and "Get started" sections across all side quests will help learners know what to expect.

Adding code snippets to the Summary sections is particularly useful - makes them work as quick reference rather than just recap text.

One small note: "What's next?" uses ## while the intro subsections use ###, so the ending ends up more prominent than the beginning. Not a blocker, just flagging in case it wasn't intentional.

@vdauwera
Copy link
Collaborator Author

vdauwera commented Dec 3, 2025

Thanks @pinin4fjords ! Yep that ## was intentional, it's so the what's next stands out vs the summary.

@vdauwera vdauwera merged commit ec0f143 into master Dec 3, 2025
9 checks passed
@vdauwera vdauwera deleted the gvda-sidequests-layout branch December 3, 2025 13:57
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