-
Notifications
You must be signed in to change notification settings - Fork 0
Initial support for webpack@2. #10
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
base: master
Are you sure you want to change the base?
Conversation
This is obviously a breaking change because of the changes to the webpack API. Smoke tests have been added to ensure it works in the most basic of cases. `postcss-import` is pinned to `8.1.0` because of a breaking bug with `webpack@2`. See: postcss/postcss-import#207. Support for `node@0.12` is removed because `webpack@2` does not support it.
4552e55 to
e6d4efc
Compare
baer
left a comment
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.
🎉
| # Use GCC 4.8 if it's available | ||
| - 'if [ ! `which gcc-4.8` == "" ]; then export CXX=g++-4.8 && export CC=gcc-4.8; fi' | ||
| - node_js: 4 | ||
| - node_js: 5 |
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.
Today you can drop 5!
| styleLoader, | ||
| `${pack(cssLoader, config)}!${loader}` | ||
| ); | ||
| return [plugin.loader({omit: 1, remove: true}), { |
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.
What is this about?
| }), | ||
| ...(Array.isArray(options) ? options : options(webpack)), | ||
| autoprefixer({ | ||
| browsers: ['last 2 versions'], |
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.
Should this be configurable?
This is obviously a breaking change because of the changes to the webpack API. Smoke tests have been added to ensure it works in the most basic of cases.
postcss-importis pinned to8.1.0because of a breaking bug withwebpack@2. See: postcss/postcss-import#207.With
webpack@2locking down properties on theloaderobject it's no longer possible to merge "same" loaders withwebpack-partial.Support for
node@0.12is removed becausewebpack@2does not support it.See:
optionsto children. webpack-contrib/extract-text-webpack-plugin#266optionsviaquery. webpack/postcss-loader#104/cc @nealgranger @baer