Conversation
evgdudareff
commented
Jan 29, 2020
evgdudareff
commented
- режимы dev, build
- минификация файлов
- babel
-relocate all files into src folder -add dev and build srcipts -add minification for files -ie 11 doesn`t work :(
Fix babel loader config
gooverdian
left a comment
gooverdian
left a comment
There was a problem hiding this comment.
Можно чуть улучшить, в целом - очень хорошо - пока оценка 7 / 10
package.json
Outdated
| "postcss-loader": "^3.0.0", | ||
| "style-loader": "^1.1.3", | ||
| "terser-webpack-plugin": "^2.3.3", | ||
| "url-loader": "^3.0.0" |
There was a problem hiding this comment.
почти все из этого должно быть в dev, незачем захламлять production сборку
если нет понимания что куда:
production - то, без чего не заработает уже собранное
development - все остальное - то, без чего не соберется, или то, что нужно для разработки (дев тулзы)
There was a problem hiding this comment.
У меня получилось, что всё нужно только при сборке. Перенес
| ]; | ||
| } | ||
| return config; | ||
| }; |
There was a problem hiding this comment.
Эту функцию можно переписать проще, возвратив пустой объект в начале
webpack.config.js
Outdated
| module.exports = { | ||
| devtool: "source-map", //связывает минимизированный и собранный код с исходным | ||
| context: path.resolve(__dirname, "src"), //относительный путь | ||
| mode: "development", //режим работы webpack по умолчанию |
There was a problem hiding this comment.
Если что-то сломается - ты выкатишь development версию на прод. По умолчанию всегда должно быть production
webpack.config.js
Outdated
| }; | ||
|
|
||
| module.exports = { | ||
| devtool: "source-map", //связывает минимизированный и собранный код с исходным |
There was a problem hiding this comment.
Сам факт использования сурс-мапов - отличная идея, но нужны ли сурс-мапы для development?
There was a problem hiding this comment.
Прошу прощения, имел в виду production, конечно
There was a problem hiding this comment.
Теперь сурсы только в development
| } | ||
| }), | ||
| //очищает папку dist | ||
| new CleanWebpackPlugin(), |
| } | ||
| ]), | ||
| //минимизация картинок | ||
| new ImageminWebpackPlugin({ |
- relocated packages to devDependencies; - simplified optimize function; - set production mode by default; - source-map active only for development mode
gooverdian
left a comment
gooverdian
left a comment
There was a problem hiding this comment.
Да, так лучше
9 / 10
(это максимальный балл без галпа, на нем я не настаиваю, но если кто-то сделает, у меня должна быть возможность это оценить)
|
|
||
| //Функция оптимизации | ||
| const optimize = () => { | ||
| if (!isProduction) return {}; |
There was a problem hiding this comment.
Сейчас у вас нет codestyle-guide, и ты можешь писать, как хочешь. Но на заметку - почти во всех стайлгайдах запрещена такая запись, всегда нужно использовать фигурные скобки, это повышает читаемость кода, я имел в виду примерно такой вид функции:
const optimize = () => {
if (isDevelopment) {
return {};
}
return {
minimizer: [new OptimizeCssAssetsPlugin(), new TerserWebpackPlugin()],
};
};2 participants
