Skip to content

Add a build verdict job - #292

Open
bkhouri wants to merge 1 commit into
swiftlang:mainfrom
bkhouri:t/main/add_required
Open

Add a build verdict job#292
bkhouri wants to merge 1 commit into
swiftlang:mainfrom
bkhouri:t/main/add_required

Conversation

@bkhouri

@bkhouribkhouri commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Add a build verdict job that will fail when any dependent workflow fails. The workflow also emits, on the console, all required job names that failed.

Depends on #290

@bkhouribkhouri changed the title T/main/add required[WIP] Add a Required job that depends on all othersJun 26, 2026
@bkhouri
bkhouriforce-pushed the t/main/add_required branch 4 times, most recently from 4ea8872 to 97ec0d6CompareJuly 2, 2026 19:56
@bkhouribkhouri changed the title [WIP] Add a Required job that depends on all othersAdd a Required job that depends on all othersJul 23, 2026
@bkhouri
bkhouri marked this pull request as ready for review July 23, 2026 19:15
@bkhouri
bkhouri requested a review from a team as a code ownerJuly 23, 2026 19:15
@bkhouri
bkhouriforce-pushed the t/main/add_required branch 2 times, most recently from b162bae to 367a465CompareJuly 23, 2026 19:19
Add a build verdict job that will fail when any dependent workflow
fails. The workflow also emits, on the console, all required job names
that failed.
@bkhouri
bkhouriforce-pushed the t/main/add_required branch from 367a465 to 1a67e09CompareJuly 23, 2026 19:20
@bkhouribkhouri changed the title Add a Required job that depends on all othersAdd a build verdict jobJul 23, 2026

@FranzBuschFranzBusch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So this is only for this repo. I would like us to discuss and agree if this is expected from all repos that depend on this one as a guidance.

Sign up for freeto 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

@bkhouri@FranzBusch