Skip to content

Commit 66c1d49

Browse files
authored
Add note about conflicting meaning of input assignment
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
1 parent 622fabe commit 66c1d49

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

adr/20260312-record-syntax-unification.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ process FASTQC {
152152
- Good, because `record()` is already a general-purpose function, no new syntax needed.
153153
- Good, because type annotations follow standard rules — `sample: Sample = record(...)` works like any typed assignment.
154154
- Bad, because input `record()` arguments are types rather than values, which is a different usage of the function.
155-
155+
- Bad, because the use of an assignment in the input declaration to describe the record structure conflicts with the conventional meaning of input assignments as default values.
156156
## Solution or decision outcome
157157

158158
**Option 3**: Use the `record()` function-call notation uniformly for both process inputs and outputs, combined with standard assignment (`=`) and optional type annotation (`: Type`).

0 commit comments

Comments
 (0)