AM
array_merge - та ещё штучка...
Size: a a a
2020 June 19
NO
Как именно станет легче? Если писать будет КАЖДЫЙ раз?
DS
Вот эту часть я и хотел как раз отдельно сделать
Вынести ее в отдельный класс из класса Plugin и вызывать в нем ConfigCollector->collect и Builder->build
AM
Вынести ее в отдельный класс из класса Plugin и вызывать в нем ConfigCollector->collect и Builder->build
Ну, в принципе, норм.
AM
Как именно станет легче? Если писать будет КАЖДЫЙ раз?
Наверное, никак :)
NO
Тогда повторяю вопрос, зачем писать на диск?
AM
Ну, например, посмотреть какой конфиг получается.
AM
Потому что в проде лучше гонять из написанного на диск.
AM
Плюс можно как раз писать на диск и коммитить в репку.
DS
Тогда повторяю вопрос, зачем писать на диск?
Это файловый кеш, в проде все берется из него и не ребилдитс каждый раз
NO
Это файловый кеш, в проде все берется из него и не ребилдитс каждый раз
Речь-то не про прод.
AM
В дебаг-режиме ребилдится каждый раз.
NO
Плюс можно как раз писать на диск и коммитить в репку.
Подозреваю, что в конфиг могут попасть пароли, в таком случае не стоит его коммитить...
AM
Подозреваю, что в конфиг могут попасть пароли, в таком случае не стоит его коммитить...
Если getenv() юзать, то нет.
NO
Ну, например, посмотреть какой конфиг получается.
Можно сделать опцией.
NO
То есть упирается в то как написан плагин ) По-другому он не умеет?
NO
Спасибо за issue )
AM
Ну да.
AM
Чисто технического плана проблема.