Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Sep 11, 2023

Newer versions of the babel packages allow for removing some types packages as well as some helper packages. The @babel/template package export used within the CLI is accessible from the @babel/core package which allows removal of @babel/template as a direct dependency. Also, the @babel/plugin-proposal-async-generator-functions package has been transitioned to @babel/plugin-transform-async-generator-functions due to async generators being merged into the ECMAScript standard. Minor code cleanup based on the type cleanup was also performed in the build optimizer babel passes.

@clydin clydin added the target: major This PR is targeted for the next major release label Sep 11, 2023
@clydin clydin force-pushed the build/babel-packages branch from d8b6237 to ba163d2 Compare September 11, 2023 15:24
@clydin clydin changed the title refactor(@angular-devkit/build-angular): update package usage and types based on current versions refactor(@angular-devkit/build-angular): update babel package usage and types based on current versions Sep 11, 2023
@clydin clydin force-pushed the build/babel-packages branch from ba163d2 to 77ac300 Compare September 11, 2023 15:59
…nd types based on current versions

Newer versions of the babel packages allow for removing some types packages as well as some helper
packages. The `@babel/template` package export used within the CLI is accessible from the `@babel/core`
package which allows removal of `@babel/template` as a direct dependency. Also, the `@babel/plugin-proposal-async-generator-functions`
package has been transitioned to `@babel/plugin-transform-async-generator-functions` due to async generators
being merged into the ECMAScript standard. Minor code cleanup based on the type cleanup was also performed
in the build optimizer babel passes.
@clydin clydin force-pushed the build/babel-packages branch from 77ac300 to 293f3b0 Compare September 11, 2023 16:01
@clydin clydin requested a review from alan-agius4 September 11, 2023 17:12
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Sep 11, 2023
@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 Sep 11, 2023
@alan-agius4 alan-agius4 merged commit b4a12a9 into angular:main Sep 11, 2023
@clydin clydin deleted the build/babel-packages branch September 11, 2023 18:47
@angular-automatic-lock-bot
Copy link

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 Oct 12, 2023
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 target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants