Skip to content

Conversation

JeffreyThiessen
Copy link
Member

@JeffreyThiessen JeffreyThiessen commented Aug 27, 2025

What does this PR do and why?

Describe in detail what your merge request does and why.
Continues work started in #1163
Cleans up the flow of code so that jobs that receive a false return value from services handle the errors correctly.
Adds tests that were missed when the jobs were originally created.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other pull requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

This comment has been minimized.

@JeffreyThiessen JeffreyThiessen marked this pull request as ready for review August 28, 2025 19:52
@JeffreyThiessen JeffreyThiessen self-assigned this Aug 28, 2025
@JeffreyThiessen JeffreyThiessen added bug Something isn't working ready for review Pull request is ready for review labels Aug 28, 2025
@JeffreyThiessen JeffreyThiessen force-pushed the refactor_workflow_execution_jobs_part_2 branch from bac685c to be39154 Compare October 15, 2025 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ready for review Pull request is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant