Skip to content

Commit 43f75d9

Browse files
pinin4fjordsclaude
andcommitted
Prioritize visual output verification over command inspection
Changed testing section to check visual output first (faster and more natural for workshops), with command file inspection as an optional follow-up for those who want to see implementation details. Addresses PR feedback about making workshop experience more efficient. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 35b51fb commit 43f75d9

File tree

1 file changed

+21
-17
lines changed

1 file changed

+21
-17
lines changed

docs/hello_nf-core/04_make_module.md

Lines changed: 21 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -379,23 +379,7 @@ The pipeline should run successfully. In the output, look for the cowpy process
379379
[bd/0abaf8] CORE_HELLO:HELLO:cowpy [100%] 1 of 1 ✔
380380
```
381381

382-
Now let's verify that the `ext.args` configuration actually passed the character argument to the cowpy command. Use the task hash (the `bd/0abaf8` part) to inspect the `.command.sh` file in the work directory:
383-
384-
```bash
385-
cat work/bd/0abaf8*/.command.sh
386-
```
387-
388-
You should see the cowpy command with the `-c cow` argument:
389-
390-
```console title="Output"
391-
#!/usr/bin/env bash
392-
...
393-
cat test.txt | cowpy -c kosh > cowpy-test.txt
394-
```
395-
396-
This confirms that `task.ext.args` successfully passed the character parameter through the configuration rather than requiring it as a process input.
397-
398-
We can also check the output:
382+
Let's verify that the `ext.args` configuration worked by checking the output. Use the task hash (the `bd/0abaf8` part) to look at the output file:
399383

400384
```bash
401385
cat work/bd/0abaf8*/cowpy-test.txt
@@ -423,6 +407,26 @@ cat work/bd/0abaf8*/cowpy-test.txt
423407
| |
424408
```
425409

410+
You should see the ASCII art displayed with the kosh character, confirming that the `ext.args` configuration worked!
411+
412+
!!! note "Optional: Inspect the command file"
413+
414+
If you want to see exactly how the configuration was applied, you can inspect the `.command.sh` file:
415+
416+
```bash
417+
cat work/bd/0abaf8*/.command.sh
418+
```
419+
420+
You'll see the cowpy command with the `-c kosh` argument:
421+
422+
```console
423+
#!/usr/bin/env bash
424+
...
425+
cat test.txt | cowpy -c kosh > cowpy-test.txt
426+
```
427+
428+
This shows that the `.command.sh` file was generated correctly based on the `ext.args` configuration.
429+
426430
### 1.3. Add configurable output naming with ext.prefix
427431

428432
There's one more nf-core pattern we can apply: using `ext.prefix` for configurable output file naming.

0 commit comments

Comments
 (0)