-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial support for webpack@2. #10
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 `[email protected]` is removed because `webpack@2` does not support it.
4552e55
to
e6d4efc
Compare
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-import
is pinned to8.1.0
because of a breaking bug withwebpack@2
. See: postcss/postcss-import#207.With
webpack@2
locking down properties on theloader
object it's no longer possible to merge "same" loaders withwebpack-partial
.Support for
[email protected]
is removed becausewebpack@2
does not support it.See:
options
to children. webpack-contrib/extract-text-webpack-plugin#266options
viaquery
. webpack-contrib/postcss-loader#104/cc @nealgranger @baer