FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(core): Ensure JIT components also use `OnPush` by default. by JeanMeche · Pull Request #69898 · angular/angular · GitHub

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 commented Jul 22, 2026
edited
Loading

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 Bot added the area: core Issues related to the framework runtime label Jul 22, 2026
ngbot Bot added this to the Backlog milestone Jul 22, 2026
JeanMeche force-pushed the onpush-jit branch 3 times, most recently from 19ab2fb to 11a22d2 Compare July 22, 2026 23:38
JeanMeche requested a review from alxhub July 26, 2026 16:14
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.

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.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL