Skip to content

FxCI: Skip shippable builds for PRs ; add tests#303

Merged
lissyx merged 2 commits intoenterprise-mainfrom
enterprise-shippable_pgo
Jan 13, 2026
Merged

FxCI: Skip shippable builds for PRs ; add tests#303
lissyx merged 2 commits intoenterprise-mainfrom
enterprise-shippable_pgo

Conversation

@lissyx
Copy link
Contributor

@lissyx lissyx commented Dec 19, 2025

No description provided.

@lissyx lissyx requested a review from a team December 19, 2025 19:37
@lissyx lissyx marked this pull request as draft December 19, 2025 19:37
@lissyx lissyx force-pushed the enterprise-shippable_pgo branch from 704df8f to 737e5ab Compare January 9, 2026 08:11
@lissyx lissyx force-pushed the enterprise-shippable_pgo branch from 737e5ab to 258b1f2 Compare January 9, 2026 08:12
@lissyx lissyx force-pushed the enterprise-shippable_pgo branch from 258b1f2 to 6423793 Compare January 9, 2026 10:32
@lissyx lissyx marked this pull request as ready for review January 9, 2026 12:03
@lissyx lissyx requested a review from ahal January 9, 2026 12:03
@lissyx lissyx changed the title FxCI: Skip shippable builds for PRs FxCI: Skip shippable builds for PRs ; add tests Jan 9, 2026
@lissyx lissyx force-pushed the enterprise-shippable_pgo branch from 94758de to 250e9c4 Compare January 9, 2026 23:48
@lissyx lissyx force-pushed the enterprise-shippable_pgo branch from 250e9c4 to b628a6c Compare January 10, 2026 00:27
Copy link
Member

@ahal ahal left a comment

Choose a reason for hiding this comment

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

Looks reasonable! Main issue is to remove the worker-type hack.

@lissyx
Copy link
Contributor Author

lissyx commented Jan 13, 2026

Somehow we dont run tests on debug builds?

@gcp
Copy link
Contributor

gcp commented Jan 13, 2026

Somehow we dont run tests on debug builds?

We don't, for cost reasons (it's much slower on CI). This means, unfortunately, that's not uncommon to start crashing on an unrelated thing when you make a debug build to debug something...

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