Skip to content

Commit

Permalink
feat: Explicitly disallow capture_stdout option on sequence tasks (#261)
Browse files Browse the repository at this point in the history
  • Loading branch information
nat-n authored Nov 24, 2024
1 parent dde728d commit 0a9526c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion docs/tasks/task_types/sequence.rst
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ By default the contents of the array are interpreted as references to other task
Available task options
----------------------

``sequence`` tasks support all of the :doc:`standard task options <../options>` with the exception of ``use_exec``.
``sequence`` tasks support all of the :doc:`standard task options <../options>` with the exception of ``use_exec`` and ``capture_stdout``.

The following options are also accepted:

Expand Down
4 changes: 4 additions & 0 deletions poethepoet/task/sequence.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,10 @@ def validate(self):
"Unsupported value for option `default_item_type`,\n"
f"Expected one of {PoeTask.get_task_types(content_type=str)}"
)
if self.capture_stdout is not None:
raise ConfigValidationError(
"Unsupported option for sequence task `capture_stdout`"
)

class TaskSpec(PoeTask.TaskSpec):
content: list
Expand Down

0 comments on commit 0a9526c

Please sign in to comment.