А
Там контейнер надо проще сделать без мокера внутри
Size: a a a
2020 August 22
А
И не все кейсы в тестах
СП
Не пойму, а как это с моим PR добавляющим подтверждение в migrate/up и migrate/down связано? Или речь про какой-то другой PR?
А
Не пойму, а как это с моим PR добавляющим подтверждение в migrate/up и migrate/down связано? Или речь про какой-то другой PR?
Ааа, это вроде норм, я разве не апрувнул?
А
Я про более ранний
СП
Ааа, это вроде норм, я разве не апрувнул?
С ним всё норм, да)
А
42й
СП
аа... понял
СП
Но они вроде с новым PR не пересекаются :)
А
Но они вроде с новым PR не пересекаются :)
Я стараюсь быть последовательным )
А
Добавишь PR на кодревью?
А, только сейчас понял, что ты теги/метки просишь
А
Сначала подумал, что мне выкатить свой пр надо 😁
СП
ааа)))
СП
Не, просто кодревью запросить для #51. Если норм всё пройдёт, чтобы слить.
СП
Спасибо
А
Не, просто кодревью запросить для #51. Если норм всё пройдёт, чтобы слить.
ты уверен, что параметры SpiralFactoryInterface::make() совместимы с параметрами фабрики?
СП
ты уверен, что параметры SpiralFactoryInterface::make() совместимы с параметрами фабрики?
Если я правильно понял, то да... Сейчас ещё раз посмотрю...
А
Если я правильно понял, то да... Сейчас ещё раз посмотрю...
в спирали вроде именованые параметры без позиционных или без автоопределения по типу
А
так что потенциально подходит. Я ревью отписал
СП
так что потенциально подходит. Я ревью отписал
А разве используя фабрику мы зависим от реализации контейнера?