Sitelet https://github.com/angular/angular/pull/69898
Skip to content

fix(core): Ensure JIT components also use OnPush by default. - #69898

Open
JeanMeche wants to merge 1 commit into
angular:mainfrom
JeanMeche:onpush-jit
Open

fix(core): Ensure JIT components also use OnPush by default.#69898
JeanMeche wants to merge 1 commit into
angular:mainfrom
JeanMeche:onpush-jit

Conversation

@JeanMeche

@JeanMeche JeanMeche commented Jul 22, 2026

Copy link
Copy Markdown
Member

Unfortunate oversight when landing #67687, it was missed here that OnPush should also be the default for JIT compiled component.

This behavior also affects tests that run in JIT mode (which is the default with Karma)

@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Jul 22, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jul 22, 2026
@JeanMeche
JeanMeche force-pushed the onpush-jit branch 3 times, most recently from 19ab2fb to 11a22d2 Compare July 22, 2026 23:38
@JeanMeche
JeanMeche requested a review from alxhub July 26, 2026 16:14
@JeanMeche
JeanMeche marked this pull request as ready for review July 26, 2026 16:14
Unfortunate oversight when landing angular#67687, it was missed here that OnPush should also be the default for JIT compiled component.
@JeanMeche

Copy link
Copy Markdown
Member Author

Marking as breaking change because of the test impacts.

Note: This shouldn't be an issue when running the tests with Vitest, as it uses AOT by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: core Issues related to the framework runtime breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JIT (TestBed) still defaults changeDetection to Eager, AOT correctly defaults to OnPush

1 participant