Skip to content

Skip rather than suicide bake task when there are no recipes#1904

Merged
daflack merged 1 commit intomainfrom
1704_skip_rather_than_suicide
Feb 17, 2026
Merged

Skip rather than suicide bake task when there are no recipes#1904
daflack merged 1 commit intomainfrom
1704_skip_rather_than_suicide

Conversation

@jfrost-mo
Copy link
Member

@jfrost-mo jfrost-mo commented Feb 10, 2026

This sets the run mode to skip instead of suicide triggering the bake task. This avoids using the semi-deprecated suicide triggers, leaves the workflow graph intact, and simplifies the code.

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

This sets the run mode to skip instead of suicide triggering the bake
task. This avoids using the semi-deprecated suicide triggers, leaves the
workflow graph in tact, and simplifies the code.
@jfrost-mo jfrost-mo self-assigned this Feb 10, 2026
@jfrost-mo jfrost-mo added the cleanup Non-functional improvement label Feb 10, 2026
@jfrost-mo jfrost-mo marked this pull request as ready for review February 10, 2026 16:19
@github-actions
Copy link
Contributor

Coverage

@jfrost-mo jfrost-mo added this to the CSET v26.2.0 milestone Feb 16, 2026
Copy link
Collaborator

@daflack daflack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy with changes, having discussed the code yesterday so that I understood what it was doing.

@daflack daflack merged commit 35c6bb3 into main Feb 17, 2026
8 checks passed
@daflack daflack deleted the 1704_skip_rather_than_suicide branch February 17, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Non-functional improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants