В
В общем, в PR я тред закрыл и открыл вместо него issue: https://github.com/yiisoft/yii-queue/issues/53
Size: a a a
2020 July 28
В
Если изменение действительно нужно - не проблема его сделать, но, ИМХО, это должен быть другой PR.
В
(но мне кажется, без него лучше) 😄
Д
Если изменение действительно нужно - не проблема его сделать, но, ИМХО, это должен быть другой PR.
да, без проблем.
Д
Не согласен с тобой: в
yii-console тогда появятся доступные к запуску команды, которые используют код, которого в этом пакете нет и быть не может. Мне больше нравится идея перенести yii-console из require-dev в require часть composer.json.да, я что-то провтыкал вчера.
это же команды чисто для yii-queue, а yii-console может и без очередей жить.
ничего выносить не нужно, пойду закрою issue.
это же команды чисто для yii-queue, а yii-console может и без очередей жить.
ничего выносить не нужно, пойду закрою issue.
В
да, я что-то провтыкал вчера.
это же команды чисто для yii-queue, а yii-console может и без очередей жить.
ничего выносить не нужно, пойду закрою issue.
это же команды чисто для yii-queue, а yii-console может и без очередей жить.
ничего выносить не нужно, пойду закрою issue.
Благодарю 🙏
В
https://github.com/yiisoft/yii-queue/pull/52#discussion_r461041469
а что по этому вопросу?
а что по этому вопросу?
Оно там действительно не нужно, удалил
В
Будем разрешать только в конструктор передавать обработчики
Д
Будем разрешать только в конструктор передавать обработчики
угу. вечером пройдусь еще раз гляну, если это терпит.
В
Вообще не горит :)
Д
а драйвера не нужно будет править?
В
а драйвера не нужно будет править?
У нас только
yii-queue-amqp, я в нем уже pr сделал. Встроенный синхронный - не надо.В
В
минутка юмора
Д
😂😂
А
milestones конечно было бы круто добавить
А
но у нас и в репозиториях их особо и нет )