Skip to content

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

Closed
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:deprecate/platform-browser-dynamic
Closed

refactor(platform-browser): Deprecate the platform-browser-dynamic package#61043
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:deprecate/platform-browser-dynamic

Conversation

@JeanMeche
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@alxhub
Copy link
Copy Markdown
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
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jun 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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