В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

Yii Framework 3

494 membersпожаловаться на группу
2020 September 25

AV

Andrii Vasyliev in Yii Framework 3
я за, потому что баг в любом случае уже там
если таки надо фиксить - то надо :)
источник
15:30пожаловаться#1

DS

Dmitriy S in Yii Framework 3
Andrii Vasyliev
я за, потому что баг в любом случае уже там
если таки надо фиксить - то надо :)
👍 оки
источник
15:31пожаловаться#2

AM

Alexander Makarov in Yii Framework 3
@hiqsol а проверку которая была ты не выпилил?
источник
15:38пожаловаться#3

AM

Alexander Makarov in Yii Framework 3
https://github.com/yiisoft/di/pull/158/files
GitHub
Fix circular reference when nested container defined by hiqsol · Pull Request #158 · yiisoft/di
First of all any closure definition for ContainerInterface like the
following IS the circular reference! Cause ContainerInterface is defined
using itself.
   ContainerInterface::class => st...
источник
15:38пожаловаться#4

AM

Alexander Makarov in Yii Framework 3
if ($id === ContainerInterface::class) {
    return $this;
}
источник
15:38пожаловаться#5

AV

Andrii Vasyliev in Yii Framework 3
она нужна
источник
15:38пожаловаться#6

AM

Alexander Makarov in Yii Framework 3
м... ок
источник
15:39пожаловаться#7

AM

Alexander Makarov in Yii Framework 3
тесты вроде похожи
источник
15:39пожаловаться#8

AM

Alexander Makarov in Yii Framework 3
сливаю
источник
15:39пожаловаться#9

AV

Andrii Vasyliev in Yii Framework 3
я в ПРе подробно расписал
источник
15:39пожаловаться#10

AM

Alexander Makarov in Yii Framework 3
там про то, что добавилось
источник
15:39пожаловаться#11

AM

Alexander Makarov in Yii Framework 3
но не про то, что уже было
источник
15:39пожаловаться#12

AM

Alexander Makarov in Yii Framework 3
влил
источник
15:39пожаловаться#13

AM

Alexander Makarov in Yii Framework 3
https://github.com/yiisoft/di/pull/156 - норм тест?
GitHub
Add test for changing `ContainerInterface` in container by hiqsol · Pull Request #156 · yiisoft/di
Q
A




Is bugfix?
❌


New feature?
❌


Breaks BC?
❌
источник
15:40пожаловаться#14

AV

Andrii Vasyliev in Yii Framework 3
источник
15:40пожаловаться#15

AM

Alexander Makarov in Yii Framework 3
@yiiliveext или нет?
источник
15:40пожаловаться#16

AM

Alexander Makarov in Yii Framework 3
Andrii Vasyliev
Это я понял. Тут не написано что он тоже нужен :)
источник
15:41пожаловаться#17

AV

Andrii Vasyliev in Yii Framework 3
Alexander Makarov
https://github.com/yiisoft/di/pull/156 - норм тест?
GitHub
Add test for changing `ContainerInterface` in container by hiqsol · Pull Request #156 · yiisoft/di
Q
A




Is bugfix?
❌


New feature?
❌


Breaks BC?
❌
это дублирует #158, не надо
источник
15:41пожаловаться#18

AV

Andrii Vasyliev in Yii Framework 3
закрыл
источник
15:42пожаловаться#19

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
@yiiliveext или нет?
Что ты хочешь спросить? Правильній ли код? - Нет, неправильный. Можно ли его временно смержить? Да, ок, даже нужно, поскольку смержили уже несколько непродуманных пр @hiqsol. В целом принцип прост и логичен, мержишь все от @hiqsol, пока я не докажу несостоятельность его изменений)
источник
15:44пожаловаться#20