phpcs и phpstan отсеивают изначально лишнее

при чем тут отсеивание. я тебе говорю о том, что когда делаю ревью, то хочу видеть контекст, а не только изминение

да, можно запустить проверки разные, но они тупые. они не поймут что return $lel ? $this->lol() : $this->lel() надо поменять местами

они смотрят только на очевидные вещи

Vladyslav а у вас все приватные репо на гитхабе?

да

ну много очень приватных

мы апсорс юзаем для этого, он делает стат анализ в интерфейсе + вообще интегрируется в шторм

так стат анализ тупой опять же

он не понимает контекст

а реагирует только на правила

вот представь ты делаешь ревью этого куска кода

вот ты первый раз его делаешь, не знаешь то делает эта функция

как тебе стат анализ поможет?

ты должен пойии в сорцы, найти, разобраться и тогда принимать решение это ок или не пускать такое

так стат анализ тупой опять же

попробуй psalm, он хотя бы типы выводить умеет

стат анализ для javascript, конечно, ничем не поможет

стат анализ для javascript, конечно, ничем не поможет

typescript, flow, все хорошо

вопрос в том есть у тебя возможности описать типы или нет

та для любого языка, стат анализ тупой, он работает по правилам. он реагирует на вот такое if (false && this.lol())

но вот саму логику он не проверит

вот для куска, который я скинул

мне кажется, ты не понимаешь, что такое статанализ...

но вот саму логику он не проверит

код ревью не для поиска багов тупых надо просто юзать. Ну то есть не только для этого

typescript, flow, все хорошо

это уже не javascript

как ты будешь делать ревью? ты понаедеешься что resolveImage() работает правильно?

добавить к этому тесты

как ты будешь делать ревью? ты понаедеешься что resolveImage() работает правильно?

для чего ты ревью делаешь?)

мне кажется, ты не понимаешь, что такое статанализ...

я все понимаю

я все понимаю

мне кажется нет)

та для любого языка, стат анализ тупой, он работает по правилам. он реагирует на вот такое if (false && this.lol())

вот такие штуки стат анализ нормальный кушает неплохо

и будет ругаться на dead code

естественно что под php с этим пока плохо (из всего что есть с этим более-менее справляется psalm но он слишком молодой пока)

мне кажется нет)

чего я не понимаю ? что я опустил и как статаанзл поможет проблеме отсутствия контекста?

статанализ анализирует контекст

чего я не понимаю ? что я опустил и как статаанзл поможет проблеме отсутствия контекста?

какого контекста ему там не хватает?)

вот ты делаешь ревью

еще раз кидаю

еще раз кидаю

ну и?

я вижу что чувак заюзал функцию из утилс

я не помню что она делает

я ему не верю

я ему не верю

твои проблемы что ты работаешь с людьми которым не веришь

ай ну вас)

я понял)

я бы написал коммент "our project should be free from utils modules"

и пошел бы дальше по своим делам

стат анализ на то и стат анализ, что статически, т.е. без исполнения, определит - что это за функция, где находится, и по желанию - даст на нее ссылку

ну да, анализ + тесты фиксят эту проблему как бы, но не совсем

стат анализ на то и стат анализ, что статически, т.е. без исполнения, определит - что это за функция, где находится, и по желанию - даст на нее ссылку

да и в тулах типа upsouce ты можешь кликнуть и перейти

ну то есть блин... я вообще не понимаю в чем проблема.... в том что для ревью используется неудобный инструмент?

ну и в пхпшторме тоже) но на гитхабе такого нет

ну так я с этого и начал, что мы юзаем апсорс, статанализ которого позволяет ходить по коду

ну и в пхпшторме тоже) но на гитхабе такого нет

подключи upsource к github

и делай ревью там

до 10-ти человек бесплатно, только мне приходится 40 баксов в месяц за сервак платить

и будешь счастлив

ты даже сможешь из phpstorm ревью делать)))

та я вот уже задумался

да прожорливая сука... как и все жавовое

https://plugins.jetbrains.com/plugin/10448-github-code-reviews-plugin

вот еще такие штуки есть но я не ресерчил

я попробую

да прожорливая сука... как и все жавовое

с меньше 8-ми гигов оперативы он тупо крашится.... а я ж не люблю инструкции перед применением читать)

ай

подключи upsource к github

Ну оно же локально а как это будет делать онлайн сервис я не очень понимаю

апсорс работает с внешними репозиториями... и только с ними

но там нет понятия "пуш реквест", к слову, его нужно как-то отдельно хуками воротить

апсорс работает с внешними репозиториями... и только с ними

когда мы говорили про статаанализ, я думал о чем-то проще. апсоурс огромный )

ну я попробую его

ну так потому и огромный, что там ide запихнули по сути туда ;)

все продукты это иде

с плагинами

Ну оно же локально а как это будет делать онлайн сервис я не очень понимаю

что локально?

upsource на сервере, github на сервере, между ними oauth,

что локально?

с кодом работает как иде ты ж сам сказал что можно в шторме открыть

с кодом работает как иде ты ж сам сказал что можно в шторме открыть

короч разберись потом задавай вопросы)

к шторму есть плагин который позволяет удаленно работать с кодом и добавлять комменты (так как локально тоже привязка к коммитам)

у меня вопросов нет просто для себя не видел никогда проблемы отревьюить в гитхаб/лаб пр

у меня вопросов нет просто для себя не видел никогда проблемы отревьюить в гитхаб/лаб пр

ну не видел - проходи мимо)

но upsource попробуй шутки ради

меня например ревью пул реквестов не особо интересует - я ревьювю мастер

и мне удобно это делать из IDE

с привязкой к ревизиям и что бы это автоматом попадало в upsource, что бы другие могли тоже все видеть и отвечать, что бы принимать решения что делать с кодом и т.д

ты его ревьювишь когда туда накидали коммитов уже? как часто?

ты его ревьювишь когда туда накидали коммитов уже? как часто?

раз в день-два

норм

а ПР кто-то ревьювит или просто сливают?

с привязкой к ревизиям и что бы это автоматом попадало в upsource, что бы другие могли тоже все видеть и отвечать, что бы принимать решения что делать с кодом и т.д

при таких раскладах я понимал что он нужен но у нас все в гитфлоу было так что не было необходимости так извращаться

при таких раскладах я понимал что он нужен но у нас все в гитфлоу было так что не было необходимости так извращаться

ну то есть вмерджили и хуй с ним)

все как мы любим

пидары

а ПР кто-то ревьювит или просто сливают?

просто сливают) стайл чек, статический анализ, тесты.... хотим еще метрики собирать что бы отслеживать кто покрытие кода уменьшил, кто связанность увеличил

ну то есть continious integration

если у тебя все раз в день коммитят в мастер то много говна они физически не успеют сделать

и говно проще фиксить чем предотвращать от возникновения

ну и там для совсем тяжелых случаев можно в парное программирование поиграть