sos: The stop/fail/warn_if series of actions
The skip option had a long history from VPT and is used to skip a section
[10: skip=True]
It was supposed to be used for
- Quickly comment on a step but renaming the step would have the same effect.
- Enable certain steps depending on the complexity of the workflow (e.g.
sos run --quality-control true) but we already have many ways to do the same (e.g. subworkflow).
I have never used it for any production code and do not see a good usage for it.
About this issue
- Original URL
- State: closed
- Created 6 years ago
- Comments: 46 (46 by maintainers)
Commits related to this issue
- Deprecate section option skip #1132 — committed to vatlab/sos by deleted user 6 years ago
- Deprecate section option skip #1132 — committed to vatlab/sos by deleted user 6 years ago
- Make fail_if terminate all other steps #1132 — committed to vatlab/sos by deleted user 5 years ago
- Implement option keep_output of stop_if #1132 — committed to vatlab/sos by deleted user 5 years ago
- Add actions done_if and skip_if #1132 — committed to vatlab/sos by deleted user 5 years ago
- Add tests for done_if and skip_if #1132 — committed to vatlab/sos by deleted user 5 years ago
Yes, that sounds better than
stop_if(no_output=True/False). We have swing between different options quite a few times now, let us finalize it this way.