Skip to content
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

Cleanup ngtools/webpack internal loaders #21295

Merged
merged 2 commits into from
Jul 7, 2021

Conversation

clydin
Copy link
Member

@clydin clydin commented Jul 6, 2021

The source file for the internal inline resource loader is moved into a loaders subdirectory for better code organization. The need to call require.resolve is also removed from the consumers of this loader.
When JIT mode and directTemplateLoading were enabled, the plugin would reference the raw-loader dependency. This dependency was not mentioned in the package.json and is also now deprecated. An internal direct resource loader is now used instead which removes the usage of the implicit dependency.

The source file for the internal inline resource loader is moved into a `loaders` subdirectory for better code organization. The need to call `require.resolve` is also removed from the consumers of this loader.
When JIT mode and `directTemplateLoading` were enabled, the plugin would reference the `raw-loader` dependency. This dependency was not mentioned in the `package.json` and is also now deprecated.
An internal `direct-resource` loader is now used instead which removes the usage of the implicit dependency.
@google-cla google-cla bot added the cla: yes label Jul 6, 2021
@clydin clydin marked this pull request as ready for review July 6, 2021 18:15
@clydin clydin requested a review from alan-agius4 July 6, 2021 20:15
@clydin clydin added the target: minor This PR is targeted for the next minor release label Jul 6, 2021
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Jul 7, 2021
@clydin clydin merged commit 5256699 into angular:master Jul 7, 2021
@clydin clydin deleted the ngtools/restructure-1 branch July 7, 2021 13:24
@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 Aug 7, 2021
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: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants