Closed ikru closed 7 years ago
Hi Ivan,
Thanks! Will take a look later this week. As the config approach is offered to be changed as we discussed, got to go carefully through these changes.
Так.. чертов конфиг.. в общем, я закрутил все что хотел.. есть папка config/extend в нее пользователь может складывать конфиги в соотвествии с правилами node-config и они будут мерджиться с основным конфигом. Мне не нравиться, как ты вытаскиваешь какие-то непонятные объекты в gulp.config.. как будто специфические для функций но вынесенные куда-то отдельно.. думаю, что то что конфигурация, надо пихнуть в конфиг, а объекты которые нужны для модулей вызываемых из галпа надо прям в галпе и формировать.. когда смерджишь и заставишь работать, посмотри, я думаю поймешь что я имею ввиду. Действительно потерял проверку пароля и отдельных параметров, но теоретически ситуация редкая, в общем можно будет потом из твоего старого файла выдернуть.
И надеюсь мои коммиты от сегодня будут частью этого пул реквеста...
Sorry guys, still can't merge this in a proper way as I want to do it.
This module is used in tons of our own project setups so we check double before changing it now.
Your idea will be definitely applied soon:
Please stay tuned. Thank you for the patience.
I was able to keep all existing functionality, however minor discrepancies are possible...