CI
У дата саенса нет кодревью, black + flake8
есть
Size: a a a
2019 August 14
SO
Датасайнтисты это просто математики (как правило только статистика) которые могут что-то написать на R или питоне в лучшем случае, обычно в компаниях такие сидят в отделах RnD и в прод не пишут, а рабочий код пишут уже датаинженеры и вот там процедуру кодревью вводить крайне желательно
CI
Датасайнтисты это просто математики (как правило только статистика) которые могут что-то написать на R или питоне в лучшем случае, обычно в компаниях такие сидят в отделах RnD и в прод не пишут, а рабочий код пишут уже датаинженеры и вот там процедуру кодревью вводить крайне желательно
обычно, но бывают исключения
OF
Это там, где денег много :). У нас все на все руки мастера
SO
Это там, где денег много :). У нас все на все руки мастера
Тогда вряд-ли у вас много времени на код ревью есть вообще)
OF
Стандартизация, линтеры спасают )
CI
Стандартизация, линтеры спасают )
линтеры не спасают от логики)
CI
у нас, например, есть и код ревью и полное покрытие юнит тестами
OF
Тут тесты и практика, да.
OF
Но в первую очередб максимальное упрощение.
CI
ну, когда надо MVP какое-нибудь
CI
потом постепенно наращивается функционал по мере подтверждения гипотез
CI
ничего не делается просто так без подтверждения гипотез
AK
у нас, например, есть и код ревью и полное покрытие юнит тестами
полное это как?)
CI
полное это как?)
ну каждая строчка кода тестируется)
CI
тесты тоже можно криво написать)) которые ничего не будут тестить, но вызовут каждую строчку кода)
V
Датасайнтисты это просто математики (как правило только статистика) которые могут что-то написать на R или питоне в лучшем случае, обычно в компаниях такие сидят в отделах RnD и в прод не пишут, а рабочий код пишут уже датаинженеры и вот там процедуру кодревью вводить крайне желательно
попрошу
V
коде-ревью нужно везде
только каждый его делает под проекты немножко по-разному
только каждый его делает под проекты немножко по-разному
V
у нас, например, есть и код ревью и полное покрытие юнит тестами
а как именно делаете коде-ревью?
CI
а как именно делаете коде-ревью?
двое человек просматривают код и тесты. Делают свои замечания. Человек все исправляет. Как только получено два апрува, то код мержится в мастер