Skip to content

chore(npm): updated webpack version and config cleanup #2237

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

Conversation

TheLarkInn
Copy link
Member

@TheLarkInn TheLarkInn commented Sep 20, 2016

  • webpack .25 has a significant breaking changes that requires reworking of configuration

{
enforce: "left",
Copy link

Choose a reason for hiding this comment

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

preLoader -> enforce: "right"

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks I updated this

@@ -109,7 +109,7 @@
"typedoc": "^0.4.2",
"typescript": "2.0.2",
"url-loader": "^0.5.7",
"webpack": "2.1.0-beta.22",
"webpack": "^2.1.0-beta.23",
Copy link

Choose a reason for hiding this comment

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

better fix the version

Copy link
Member Author

Choose a reason for hiding this comment

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

Was working on that as we speak lol!

@@ -157,6 +157,7 @@
"tree-kill": "^1.0.0",
"ts-node": "^1.3.0",
"tslint": "^3.11.0",
"walk-sync": "^0.2.6"
"walk-sync": "^0.2.6",
"webpack": "^2.1.0-beta.23"
Copy link

Choose a reason for hiding this comment

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

why dependency and devDependency

Copy link
Member Author

Choose a reason for hiding this comment

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

The angular-cli creates a new project, and the repo itself is a dev-dep of the generated project. So in the scope of the angular-cli, webpack is a dependency, not a dev dependency.

Copy link
Member Author

Choose a reason for hiding this comment

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

A new project may look like this.

screen shot 2016-09-20 at 10 36 30 am

@leon
Copy link
Contributor

leon commented Sep 21, 2016

They renamed left/right to pre/post

https://github.com/webpack/webpack/releases/tag/v2.1.0-beta.24

@TheLarkInn
Copy link
Member Author

@leon thanks, I'm aware I'm on the webpack core team!!! 😅 😅 Just hadn't had the time to get back to the PR with some <3<3

@TheLarkInn
Copy link
Member Author

@hansl after careful inspection, I have been able on a test project to get this to run successfully, but have no idea what is causing the e2e test failures. One thing I have noticed is that whenever I run the e2e test runner locally I find that the builds are still running against [email protected] when they should be running against .25. See below.
screen shot 2016-09-27 at 9 27 13 am

@TheLarkInn
Copy link
Member Author

Well hold that thought @filipesilva just mentioned that there are two package.json 🔥 that I have to update this in. I'm going to try this out locally first.

@TheLarkInn TheLarkInn changed the title chore(npm): updated webpack version and reworked configs [WIP] chore(npm): updated webpack version and config cleanup Sep 27, 2016
@filipesilva filipesilva merged commit 5431abc into angular:master Sep 28, 2016
JJayet pushed a commit to JJayet/angular-cli that referenced this pull request Oct 2, 2016
* chore(npm): updated webpack version and reworked configs for breaking changes

* fix: swapped enforce rules by accident, fix this

* fix: fix version, as well as convert loaders to rules for future api changes

* chore: update webpack to beta.24

* chore(npm): updated webpack to beta.25

* chore(lint): remove rogue double quote

* Added augmented module typings for webpack v2 supporting webpack.LoaderOptionsPlugin

* chore(lint): linting chores spaces, return lines, semis

* chore(npm): updated both package.jsons cause who forgets to do that
deebloo pushed a commit to deebloo/angular-cli that referenced this pull request Oct 3, 2016
* chore(npm): updated webpack version and reworked configs for breaking changes

* fix: swapped enforce rules by accident, fix this

* fix: fix version, as well as convert loaders to rules for future api changes

* chore: update webpack to beta.24

* chore(npm): updated webpack to beta.25

* chore(lint): remove rogue double quote

* Added augmented module typings for webpack v2 supporting webpack.LoaderOptionsPlugin

* chore(lint): linting chores spaces, return lines, semis

* chore(npm): updated both package.jsons cause who forgets to do that
@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 10, 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.

5 participants