Skip to content

Commit 9808a90

Browse files
Align directive-order wording in the pointer target
Address CoPilot review on #226: the _SUPPORTED_FIXTURES comment points readers at _apply_metadata_directives for the document-order behavior, but that function's own comment still said "IN KEY ORDER" / "YAML key order" -- the sorted-by-key ambiguity this branch removes at the call site. Reword it to "document order" so both spots agree. Comment-only; no behavior change.
1 parent e82dba7 commit 9808a90

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

tests/conformance/test_observability.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1590,8 +1590,9 @@ def _apply_metadata_directives(
15901590
) -> tuple[dict[str, Any], bool]:
15911591
"""Run a node's (or per-attempt's) in-node metadata directives, returning the
15921592
resulting state update and whether the node should then raise."""
1593-
# Directives run IN KEY ORDER -- 043 augments then captures, 045 attempt 1
1594-
# captures then augments, and the YAML key order encodes that. The capture
1593+
# Directives run IN DOCUMENT ORDER -- 043 augments then captures, 045
1594+
# attempt 1 captures then augments, and the YAML document order encodes
1595+
# that. The capture
15951596
# records the read's type so the immutability invariant can verify it was a
15961597
# MappingProxyType. ``raises`` is terminal (a real node body that raises runs
15971598
# nothing after), so stop at it rather than processing later directives.

0 commit comments

Comments
 (0)