NO
ребят, объясните почему тест этот из PR не проходит) а то суть разногласий пытаюсь уловить
Size: a a a
2020 June 29
В
Шанс того, что композитный контейнер будет работать некорректно одинаков в обоих случаях.
Согласен. Только в варианте со вторым интерфейсом реализацию проще поддерживать и расширять.
В
ребят, объясните почему тест этот из PR не проходит) а то суть разногласий пытаюсь уловить
Там статический анализатор ругается на код, не имеющий отношения к PR
NO
блин
NO
теперь я ещë сильнее запутался
NO
разве тест из PR не демонстрирует суть PR?
DS
Знать бы...
На пальцах:
public function login (Psr\Di\ContainerInterface $container)
{
if ($container->hasDefinition('service') { ... }
}
public function login (Psr\Di\ContainerInterface $container)
{
if ($container->hasDefinition('service') { ... }
}
В
На пальцах:
public function login (Psr\Di\ContainerInterface $container)
{
if ($container->hasDefinition('service') { ... }
}
public function login (Psr\Di\ContainerInterface $container)
{
if ($container->hasDefinition('service') { ... }
}
И сам себе злобный буратино тот, кто так напишет перед тем, как сменить контейнер.
DS
разве тест из PR не демонстрирует суть PR?
О чем ты? Тест проходит, не проходит только статический анализ
DS
И сам себе злобный буратино тот, кто так напишет перед тем, как сменить контейнер.
Вот таких вот буратин 80%+ разработчиков
В
разве тест из PR не демонстрирует суть PR?
Статический анализатор весь код смотрит
NO
О чем ты? Тест проходит, не проходит только статический анализ
Если тест проходит, то зачем тест?
В
Вот таких вот буратин 80%+ разработчиков
has('id', 'namespace') тоже все поломает)))В
Если тест проходит, то зачем тест?
?
В
Если ты про тот тест, который @yiiliveext добавил, то он не проходит в мастере
NO
Тест по идее описывает новую функциональность. И не должен проходить пока она не добавлена.
NO
так?
В
Этот тест описывает кейс в имеющейся функциональности. И этот кейс сейчас в мастере не работает.
В
А суть прений в том, каким именно образом этот кейс пофиксить: как @yiiliveext написал в PR, или каким другим способом.
NO
Я спрашиваю, что именно тестирует этот тест) и что сделать чтобы он зафейлил