Skip to content

fix: workflow missing empty list #187

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 4, 2023
Merged

fix: workflow missing empty list #187

merged 1 commit into from
Apr 4, 2023

Conversation

jackhodgkiss
Copy link
Contributor

#172 failed when attempting to to iterate over workflows which don't exist. This isn't an issue however for caas-slurm-appliance and ansible-slurm-appliance they were missing [] which would allow Ansible to skip over the empty list as opposed to failing.

@jackhodgkiss jackhodgkiss requested a review from markgoddard April 4, 2023 12:22
@jackhodgkiss jackhodgkiss requested a review from a team as a code owner April 4, 2023 12:22
@markgoddard markgoddard merged commit 9c22add into main Apr 4, 2023
@markgoddard markgoddard deleted the fix-missing-list branch April 4, 2023 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants