А
must?
Size: a a a
2020 February 05
AM
testValueShouldBeUsedForOptionalParameter -> testValueUsedForOptionalParameter
AM
testDefaultSholdNotBeUsedForOptionalParameter -> testDefaultNotUsedForOptionalParameter
DS
С Индии товарищей надо попросить, они временами интересно тесты называют.
А
поиск по всем тестам даёт не такой уж большой зоопарк тестов с should в названии. наверное просто у нас мало тестов )
DS
must?
please
DS
Тестов мало по тем пакетам, что я смотрел, надо дописывать.
AM
Тестов мало по тем пакетам, что я смотрел, надо дописывать.
Это однозначно.
DS
О, уже смержили, не успел добавить исправления для @thenotsoft
DS
Александр Некрасов
и параметры не простовляются, если не задать их в pattern, к примеру: urlGenerate('/v1/blog/test',['test'=>'test']) если в правеле их нету, должно выдать v1/blog/test?test=test а выдает v1/blog/test
https://github.com/yiisoft/router-fastroute/pull/20
Я там с прошлого PR коммиты случайно захватил, но они должны смержиться нормально.
Посмотрите названия тестов, поисправлял немного
Я там с прошлого PR коммиты случайно захватил, но они должны смержиться нормально.
Посмотрите названия тестов, поисправлял немного
DS
А не, не нормально.
DS
Смержил
AM
откомментил по именам, сам фикс норм
DS
готово
AM
Влил 👍
RM
Я тут :)
DS
Я тут :)
Поздно, все вкусненькое уже разобрали😕
А
Поздно, все вкусненькое уже разобрали😕
есть другие вкусняшки
отревьювить https://github.com/yiisoft/yii-cycle/pull/17
отревьювить https://github.com/yiisoft/yii-cycle/pull/17
DS
Там ошибочное исправление в цикле, его не нужно применять