Skip to content

Conversation

aecz
Copy link
Contributor

@aecz aecz commented Aug 14, 2017

@ngtools/webpack requires a peer dependency of enhanced-resolve@^3.1.0. It currently relies on webpack providing this dependency and on NPM 3+ flat tree installation. Webpack could remove this dependency or NPM change the tree structure. This change also removes a NPM warning about missing peer dependency.

@ngtools/webpack requires a peer dependency of enhanced-resolve@^3.1.0. It currently relies on webpack providing this dependency and on NPM 3+ flat tree installation. Webpack could remove this dependency or NPM change the tree structure. This change also removes a NPM warning about missing peer dependency.
@filipesilva
Copy link
Contributor

Heya, I don't think webpack will move enhanced-resolve to a peer dependency anytime soon (it would require a lot of projects to add it).

So yeah, the better solution for us right now is to add it ourselves. It's better to move it from a peer dependency in @ngtools/webpack to a normal dependency. That'll fix the CI error your PR is having.

@filipesilva filipesilva self-assigned this Aug 14, 2017
@filipesilva filipesilva self-requested a review August 14, 2017 16:44
@aecz aecz requested a review from hansl as a code owner August 14, 2017 16:58
Copy link
Contributor

@filipesilva filipesilva left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@filipesilva filipesilva merged commit 0ba8bb2 into angular:master Aug 17, 2017
@aecz aecz deleted the patch-1 branch August 17, 2017 19:43
dond2clouds pushed a commit to d2clouds/speedray-cli that referenced this pull request Apr 23, 2018
…o dependencies (angular#7385)

@ngtools/webpack requires a peer dependency of enhanced-resolve@^3.1.0. It currently relies on webpack providing this dependency and on NPM 3+ flat tree installation. Webpack could remove this dependency or NPM change the tree structure. This change also removes a NPM warning about missing peer dependency.
@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 Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants