Thanks to visit codestin.com
Credit goes to github.com

Skip to content

refactor(platform-browser): Deprecate the platform-browser-dynamic package #61043

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

Closed

Conversation

JeanMeche
Copy link
Member

All entries of the @angular/platform-browser-dynamic package are now deprecated. We now recommend using the @angular/platform-browser package instead. In case a JIT application/testing environment is complaining about missing the compiler also add import '@angular/compiler'

Note: JIT compilation remains supported as before.

@pullapprove pullapprove bot requested a review from atscott April 29, 2025 16:51
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Apr 29, 2025
@ngbot ngbot bot added this to the Backlog milestone Apr 29, 2025
@JeanMeche JeanMeche force-pushed the deprecate/platform-browser-dynamic branch 2 times, most recently from bd3c332 to 4e1cc6a Compare April 29, 2025 17:32
@angular-robot angular-robot bot added the detected: deprecation PR contains a commit with a deprecation label Apr 29, 2025
Copy link
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

LGTM!!!!!!!!

reviewed-for: fw-general, public-api

@pullapprove pullapprove bot requested review from devversion and kirjs May 8, 2025 17:35
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: public-api

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate labels May 12, 2025
@AndrewKushnir AndrewKushnir added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 14, 2025
@AndrewKushnir
Copy link
Contributor

@JeanMeche it looks like there is a conflict with the main branch. Could you please rebase this PR and resolve merge conflicts when you get a chance? Thank you.

…package

All entries of the `@angular/platform-browser-dynamic` package are now deprecated.
We now recommend using the `@angular/platform-browser` package instead.
In case a JIT application/testing environment is complaining about missing the compiler also add `import '@angular/compiler'`

DEPRECATED: All entries of the `@angular/platform-browser-dynamic`
@JeanMeche JeanMeche force-pushed the deprecate/platform-browser-dynamic branch from 4e1cc6a to df6616b Compare May 14, 2025 22:14
@AndrewKushnir AndrewKushnir removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 14, 2025
@AndrewKushnir AndrewKushnir removed request for atscott and kirjs May 14, 2025 23:30
@AndrewKushnir AndrewKushnir added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label May 14, 2025
@AndrewKushnir
Copy link
Contributor

Caretaker note: docs-only change, no presubmit is required, should be safe to proceed with the merge.

@alxhub
Copy link
Member

alxhub commented May 15, 2025

This PR was merged into the repository by commit a4e2139.

The changes were merged into the following branches: main, 20.0.x

alxhub pushed a commit that referenced this pull request May 15, 2025
…package (#61043)

All entries of the `@angular/platform-browser-dynamic` package are now deprecated.
We now recommend using the `@angular/platform-browser` package instead.
In case a JIT application/testing environment is complaining about missing the compiler also add `import '@angular/compiler'`

DEPRECATED: All entries of the `@angular/platform-browser-dynamic`

PR Close #61043
@alxhub alxhub closed this in a4e2139 May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime detected: deprecation PR contains a commit with a deprecation merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants