webpack no need for ident if using complex options anymore (facebook#2430)

andykenward · morgs32 · commit 03741263fcd6 · 2017-09-01T11:28:50.000-04:00
Since `v2.2.1` of webpack the need for `ident` for complex options is no longer required. See https://webpack.js.org/guides/migrating/#complex-options
diff --git a/packages/react-scripts/config/webpack.config.dev.js b/packages/react-scripts/config/webpack.config.dev.js
@@ -217,7 +217,6 @@ module.exports = {
           {
             loader: require.resolve('postcss-loader'),
             options: {
-              ident: 'postcss', // https://webpack.js.org/guides/migrating/#complex-options
               plugins: () => [
                 require('postcss-flexbugs-fixes'),
                 autoprefixer({
diff --git a/packages/react-scripts/config/webpack.config.prod.js b/packages/react-scripts/config/webpack.config.prod.js
@@ -223,7 +223,6 @@ module.exports = {
                 {
                   loader: require.resolve('postcss-loader'),
                   options: {
-                    ident: 'postcss', // https://webpack.js.org/guides/migrating/#complex-options
                     plugins: () => [
                       require('postcss-flexbugs-fixes'),
                       autoprefixer({

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,6 @@ module.exports = {`
`217`	`217`	`{`
`218`	`218`	`loader: require.resolve('postcss-loader'),`
`219`	`219`	`options: {`
`220`		`- ident: 'postcss', // https://webpack.js.org/guides/migrating/#complex-options`
`221`	`220`	`plugins: () => [`
`222`	`221`	`require('postcss-flexbugs-fixes'),`
`223`	`222`	`autoprefixer({`
Original file line number	Diff line number	Diff line change
`@@ -223,7 +223,6 @@ module.exports = {`
`223`	`223`	`{`
`224`	`224`	`loader: require.resolve('postcss-loader'),`
`225`	`225`	`options: {`
`226`		`- ident: 'postcss', // https://webpack.js.org/guides/migrating/#complex-options`
`227`	`226`	`plugins: () => [`
`228`	`227`	`require('postcss-flexbugs-fixes'),`
`229`	`228`	`autoprefixer({`