Skip to content

fix(@schematics/angular): remove animations module from ng new app - #29618

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
crisbeto:ng-new-animations
Feb 13, 2025
Merged

fix(@schematics/angular): remove animations module from ng new app#29618
alan-agius4 merged 1 commit into
angular:mainfrom
crisbeto:ng-new-animations

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Previously the animations module was added to the ng new app, because platform-server was using it. That's no longer the case as of angular/angular#59762 so these changes remove the dependency.

@crisbeto
crisbeto force-pushed the ng-new-animations branch 5 times, most recently from f7f63eb to 78dbe18 Compare February 12, 2025 18:36
Previously the animations module was added to the `ng new` app, because `platform-server` was using it. That's no longer the case as of angular/angular#59762 so these changes remove the dependency.
@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels Feb 12, 2025
@crisbeto
crisbeto marked this pull request as ready for review February 12, 2025 19:50
@clydin
clydin requested a review from alan-agius4 February 12, 2025 19:53
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Feb 12, 2025
@alan-agius4
alan-agius4 merged commit adf4ea5 into angular:main Feb 13, 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 Mar 16, 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: @schematics/angular target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants