Д
почему Reviewers разогнали?
Кто посмотрит небольшие PR? Вроде норм замариновались)
https://github.com/yiisoft/yii-queue/pull/54
https://github.com/yiisoft/yii-queue/pull/57
Size: a a a
2020 September 03
АН
АН
в этом https://github.com/yiisoft/yii-queue/pull/54 может сделать финальными классы MemoryLimitReached, SimpleLoop, SignalLoop ?
AM
почему Reviewers разогнали?
Вернуть? Я группу применял на Yii 2 до тройки. Потом и там и там. Но Yii 2 из вас никто не смотрел. Если кто хочет — скажите.
В
Александр Некрасов
в этом https://github.com/yiisoft/yii-queue/pull/54 может сделать финальными классы MemoryLimitReached, SimpleLoop, SignalLoop ?
Зачем? На мой взгляд, они хорошо смотрятся в перспективе расширения под конкретный проект. Разве что трейт можно.
АН
Зачем? На мой взгляд, они хорошо смотрятся в перспективе расширения под конкретный проект. Разве что трейт можно.
зачем event расширять ?, а для loop есть interface
В
Какой эвент?
АН
можно getMemoryLimit и getEventDispatcher вынести в trait
В
Александр Некрасов
можно getMemoryLimit и getEventDispatcher вынести в trait
Они оттуда и взяты. В трейте это абстрактные методы, которые любой loop сможет реализовать как ему вздумается.
АН
что-то я эти методы не вижу в трейте )
Д
Вернуть? Я группу применял на Yii 2 до тройки. Потом и там и там. Но Yii 2 из вас никто не смотрел. Если кто хочет — скажите.
теперь тегать будем Yii3?
А
последнее время вроде так и тегаем
А
хотя пальцы на re сначала лезут )
AM
теперь тегать будем Yii3?
Да.
AM
Reviewers остаётся для Yii 2. Кому хочется обратно — говорите.
Д
yii3/reviewers
yii2/reviewers
?:)
yii2/reviewers
?:)
АН
yii3/reviewers
yii2/reviewers
?:)
yii2/reviewers
?:)
+ )
AM
ну можно и так, да :)