-
Notifications
You must be signed in to change notification settings - Fork 12k
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
chore(npm): updated webpack version and config cleanup #2237
Conversation
TheLarkInn
commented
Sep 20, 2016
•
edited
Loading
edited
- webpack .25 has a significant breaking changes that requires reworking of configuration
{ | ||
enforce: "left", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
preLoader -> enforce: "right"
There was a problem hiding this comment.
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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better fix the version
There was a problem hiding this comment.
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" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why dependency and devDependency
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They renamed left/right to pre/post https://github.com/webpack/webpack/releases/tag/v2.1.0-beta.24 |
@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 |
…ps://github.com/thelarkinn/angular-cli into feature/webpack_vesion_23_with_breaking_changes
@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. |
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. |
…ps://github.com/thelarkinn/angular-cli into feature/webpack_vesion_23_with_breaking_changes
…ion_23_with_breaking_changes
* 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
* 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
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |