RM
https://github.com/yiisoft/csrf/pull/3
Нужен кодревью
Size: a a a
2020 September 09
AB
Если я в пакете
yiisoft/csrf в конфиге evetns-web использую класс из yiisoft/web, но добавлять yiisoft/web в зависимости НЕ буду. Это норм? :)В require-dev его, и в suggest, по необходимости
СП
В require-dev его, и в suggest, по необходимости
в suggest думаю не надо... допустим я ставлю этот пакет отдельно - чем мне поможет установка yii-web ?
AB
в suggest думаю не надо... допустим я ставлю этот пакет отдельно - чем мне поможет установка yii-web ?
да, тут спорно, согласен
В
https://github.com/yiisoft/docs/blob/master/000-packages.md#configs-and-defaults (последний раздел этого документа)
Ребят, ну вот же дока, как надо делать. За авторством @samdark, уже обсужденная и утвержденная.
AB
require !== require-devСП
Про дев ничего не сказано :)
Стоит добавить: добавлять или нет то, что нужно только в конфиге в require-dev
Стоит добавить: добавлять или нет то, что нужно только в конфиге в require-dev
AM
Если я в пакете
yiisoft/csrf в конфиге evetns-web использую класс из yiisoft/web, но добавлять yiisoft/web в зависимости НЕ буду. Это норм? :)Да.
AB
Вопрос. А если в проекте исползьуется composer-config-plugin, но это не yii-фреймворк, то как себя будут вести general purpose packages в данном случае?
СП
Вопрос. А если в проекте исползьуется composer-config-plugin, но это не yii-фреймворк, то как себя будут вести general purpose packages в данном случае?
Точно также, конфиг плагин всё соберёт, ему без разницы yii это или нет
AB
Точно также, конфиг плагин всё соберёт, ему без разницы yii это или нет
это наверное не хорошо? вдруг у меня там какие-то свои соглашения. К примеру, мерджу не конфиги для контейнера, а просто какие-нибудь параметры
AM
Нужно добавить
yiisoft/csrf в Packagist. Кто может?✅
СП
это наверное не хорошо? вдруг у меня там какие-то свои соглашения. К примеру, мерджу не конфиги для контейнера, а просто какие-нибудь параметры
Это уже в https://t.me/composer_config_plugin - можно добавить белые/чёрные списки пакетов...
AM
Про дев ничего не сказано :)
Стоит добавить: добавлять или нет то, что нужно только в конфиге в require-dev
Стоит добавить: добавлять или нет то, что нужно только в конфиге в require-dev
Не обязательно.
DS
это наверное не хорошо? вдруг у меня там какие-то свои соглашения. К примеру, мерджу не конфиги для контейнера, а просто какие-нибудь параметры
Вроде плагин собирает только конфиги, которые есть в конфиге в composer.json самого приложения.
СП
Вроде плагин собирает только конфиги, которые есть в конфиге в composer.json самого приложения.
да, но если названия совпадут)
AM
ну тогда намёржит, да
DS
ну тогда намёржит, да
можно именовать директорию yii-config
СП
можно именовать директорию yii-config
Сейчас по-моему жёстко зашито, что из папки config брать