Conversation
ExactlyNoSense
force-pushed
the
exactlynosense/minify-config
branch
from
0a0eb9d to
54d216f
Compare
| * | ||
| * @returns {boolean} | ||
| */ | ||
| debugMode: o('debug-mode', { |
There was a problem hiding this comment.
Мне кажется просто debug лучше назвать, т.к. оно и так понятно, что это режим.
Плюс у нас есть в настройках есть
/** @override */
runtime() {
return {
...super.runtime(),
engine: this.engine(),
debug: false,
И тут бы прораститить тот дебаг в этот.
There was a problem hiding this comment.
debug из рантайма попадает в конфиг моника, а также в глобальные константы. Не получится ли так, что кто-то с этим дебагом (из runtime) подвяжется на изменение отображения или подключение других зависимостей. А я всего-лишь хочу, чтобы всё, что было в проде, просто собралось чуть подубонее для дебага.
Можно сделать как-то так: добавить параметр/переменную окружения debug-build, которая будет делать то, что сейчас debug-mode, а ещё сделать параметр/переменную окружения debug, которая будет включать debug-build а так же проставлять debug в runtime.
Т.е. можно будет либо включать весь дебаг везде сразу, либо более точно изменять только сборку проекта. Что скажешь?
There was a problem hiding this comment.
Ну в принципе да, можно сделать в конфиге
build.debug -- параметр debug-build
и debug в корне, который включает build.debug и runtime.debug
|
|
||
| #### :rocket: New Feature | ||
|
|
||
| * Added new property `debugMode` to `build` object `config/default` |
There was a problem hiding this comment.
| * Added new property `debugMode` to `build` object `config/default` | |
| * Added a new property `build.debugMode` `config/default` |
| #### :rocket: New Feature | ||
|
|
||
| * Added new property `debugMode` to `build` object `config/default` | ||
| * Added ability to disable code optimizations for debug mode `build/webpack/optimization` |
There was a problem hiding this comment.
| * Added ability to disable code optimizations for debug mode `build/webpack/optimization` | |
| * Debugging mode now disables code minimization `build/webpack/optimization` |
|
|
||
| #### :rocket: New Feature | ||
|
|
||
| * Added ability to disable code optimizations for debug mode `build/webpack/optimization` |
There was a problem hiding this comment.
| * Added ability to disable code optimizations for debug mode `build/webpack/optimization` | |
| * Debugging mode now disables code minimization |
| /* eslint-enable camelcase */ | ||
| ]; | ||
|
|
||
| if (config.build.debugMode) { |
There was a problem hiding this comment.
А оно только JS не сжимает, или CSS тоже?
По хорошему надо бы оставить все.
|
|
||
| #### :rocket: New Feature | ||
|
|
||
| * Added new property `debugMode` to `build` object |
There was a problem hiding this comment.
| * Added new property `debugMode` to `build` object | |
| * Added a new property `build.debugMode` |
3 participants
Added ability to explicitly define whether you need a code minification or not.
Now it could be done via cli argument
optimize-minifyor via env variableOPTIMIZE_MINIFY.By default code is minified in production mode and not minified in other case.