Skip to content
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

Fix visualization of multi-branch info gathering and unexecuted components test #186

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

bensze01
Copy link
Contributor

@bensze01 bensze01 commented Nov 25, 2024

Blue Ocean doesn't display nested parallel stages correctly, so instead
refactor get_branch_info and check_every_all_sh_component_will_be_run to
handle multiple branches at once.

Also separate the unexecuted components check to its own stage, because Blue Ocean doesn't display any "single threaded" steps that share a stage with a parallel step

CI runs:

Signed-off-by: Bence Szépkúti <[email protected]>
This prepares us for the refactor in the next commit.

Signed-off-by: Bence Szépkúti <[email protected]>
Blue Ocean doesn't display nested parallel stages correctly, so instead
refactor get_branch_info and check_every_all_sh_component_will_be_run to
handle multiple branches at once.

Signed-off-by: Bence Szépkúti <[email protected]>
Blue Ocean does not properly display "single-threaded" steps in any
stage that also includes a parallel step. Separate the BranchInfo
gathering into its own separate step, so the other checks are displayed
properly.

Signed-off-by: Bence Szépkúti <[email protected]>
@bensze01 bensze01 force-pushed the dev/bensze01/multi-branch-info branch from 8935999 to be28b7d Compare November 25, 2024 16:23
@bensze01 bensze01 changed the title Improve visualization of multi-branch info gathering Fix visualization of multi-branch info gathering and unexecuted components test Nov 28, 2024
@bensze01 bensze01 added the bug Something isn't working label Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request needs: review needs: reviewer priority-medium size-s Estimated task size: small (~2d)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant