Skip to content

Changed test conditions in pipeline definition#349

Merged
dennis-behm merged 2 commits into
IBM:mainfrom
M-DLB:feature/updateCIDistributedPipeline
Aug 13, 2025
Merged

Changed test conditions in pipeline definition#349
dennis-behm merged 2 commits into
IBM:mainfrom
M-DLB:feature/updateCIDistributedPipeline

Conversation

@M-DLB
Copy link
Copy Markdown
Member

@M-DLB M-DLB commented Aug 11, 2025

Changed conditions to fix bash syntax errors

Signed-off-by: Mathieu Dalbin <mathieu.dalbin@fr.ibm.com>
@M-DLB M-DLB requested a review from dennis-behm August 11, 2025 12:22
Copy link
Copy Markdown
Member

@monvora monvora left a comment

Choose a reason for hiding this comment

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

@M-DLB, There are 2 more places that need to be updated: line#265 and line#283. I cannot leave a comment to a specific line number.

@M-DLB
Copy link
Copy Markdown
Member Author

M-DLB commented Aug 13, 2025

@monvora I did some more changes, can you check again please?

Copy link
Copy Markdown
Member

@monvora monvora left a comment

Choose a reason for hiding this comment

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

All changes look good to me.

Copy link
Copy Markdown
Member

@dennis-behm dennis-behm left a comment

Choose a reason for hiding this comment

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

Thanks @M-DLB

@dennis-behm dennis-behm merged commit 5b9c46f into IBM:main Aug 13, 2025
1 check failed
@M-DLB M-DLB deleted the feature/updateCIDistributedPipeline branch August 13, 2025 12:42
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.

3 participants