Skip to content

Commit e2ee3bf

Browse files
committed
Make it clearer that set -euo pipefail is _only_ required in
called scripts, not in the job script.
1 parent be4cbd7 commit e2ee3bf

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

src/user-guide/task-implementation/job-scripts.rst

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,8 @@ Task messages are validated by:
187187
.. autoclass:: cylc.flow.unicode_rules.TaskMessageValidator
188188
:no-index:
189189

190+
.. _job-scripts.aborting-on-error:
191+
190192
Aborting Job Scripts on Error
191193
-----------------------------
192194

src/workflow-design-guide/general-principles.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,11 @@ workflow and its tasks, all task scripts should:
152152
Examples and more details `are available <https://vaneyckt.io/posts/safer_bash_scripts_with_set_euxo_pipefail/>`_
153153
for the above three ``set`` commands.
154154

155+
Inline scripts (defined in the job-script section of the
156+
workflow configuration) do not need to ``set -euo pipefail``:
157+
It is already set as part of the
158+
:ref:`job script's error handling <job-scripts.aborting-on-error>`.
159+
155160

156161
Rose Apps
157162
---------

0 commit comments

Comments
 (0)