DT
да, единолично пока
Size: a a a
2020 October 01
m
а реквест откуда? если проект один пилишь
m
если проект твой, то реквесты эт прям збс.
хотя бы кратко понимаешь что меняется.
что *вот то говно уже реализовано в другом модуле*
хотя бы кратко понимаешь что меняется.
что *вот то говно уже реализовано в другом модуле*
m
а тут имя переменной\функции вообще какой то пздц, и нахера тут var_dump и прочие радости
DT
а реквест откуда? если проект один пилишь
От помошника
m
чет не допроверяют в плане обращений к массивам, условия кривые.
мы ещё грызем за phpdoc и " if $state === 4"
что блеать 4 ?
мы ещё грызем за phpdoc и " if $state === 4"
что блеать 4 ?
DT
а тут имя переменной\функции вообще какой то пздц, и нахера тут var_dump и прочие радости
Слава богу такой лютой дичи я не встретил) больше смущают чисто дизайнерские вещи
m
Слава богу такой лютой дичи я не встретил) больше смущают чисто дизайнерские вещи
поздравляю))
m
знач срач и субьективщина ж)
DT
Там для енумов он создал base class, назвал событие archivated
DT
Местами сделал не так как я бы сделал, но очевидно что это копипаст, и вопрос типа "я чё, реально так стремно сделал?"
RS
а на 30-тник все сектор газа слушали? вот прям интересно
Что за сектор газа?
DT
Панки хой
A
чет не допроверяют в плане обращений к массивам, условия кривые.
мы ещё грызем за phpdoc и " if $state === 4"
что блеать 4 ?
мы ещё грызем за phpdoc и " if $state === 4"
что блеать 4 ?
Гипотетически подобные вещи не должны доходить до PR а отлавливаться разными статическими анализваторами.
m
да. запускала как то проверки на проекте
A
Что за сектор газа?
Ну ты блин даешь.
DT
Гипотетически подобные вещи не должны доходить до PR а отлавливаться разными статическими анализваторами.
Почему гипотетически? Это в принципе должно быть
RS
Кстати, рассказывали про вайбер. Почему там разрабу легко выгореть
RS
Ну ты блин даешь.
Шуткую, мне в вк сразу кореша закинули эту песню, как 30 стукнуло
А
Кстати, рассказывали про вайбер. Почему там разрабу легко выгореть
?