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(workspace-plugin): fail build executor process if generate-api failed #33558

Merged

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented Jan 6, 2025

Previous Behavior

build executor ends with success even if generate-api executor fails.

Example: https://github.com/microsoft/fluentui/actions/runs/12630067248/job/35195137101?pr=33550
see react-toast

This introduces leak into API changes in v9 projects

New Behavior

We fail process if generate-api fails.

Related Issue(s)

@Hotell Hotell changed the title fix(workspace-plugin): fail executor process if generate-api failed fix(workspace-plugin): fail build executor process if generate-api failed Jan 6, 2025
@github-actions github-actions bot added this to the January Project Cycle Q1 2025 milestone Jan 6, 2025
Copy link

github-actions bot commented Jan 6, 2025

📊 Bundle size report

✅ No changes found

Copy link

github-actions bot commented Jan 6, 2025

Pull request demo site: URL

@Hotell Hotell force-pushed the tools/workspace-plugin/executors/build/fix branch from 676b142 to c3f6183 Compare January 7, 2025 17:44
@Hotell Hotell marked this pull request as ready for review January 8, 2025 10:23
@Hotell Hotell requested a review from a team as a code owner January 8, 2025 10:23
@tudorpopams tudorpopams requested a review from mainframev January 8, 2025 13:05
@Hotell Hotell merged commit 1eedd05 into microsoft:master Jan 9, 2025
18 checks passed
@Hotell Hotell deleted the tools/workspace-plugin/executors/build/fix branch January 9, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants