SK
Ревью и мысли по поводу стэка высказывают все
По ревью можно отдельно всхоливарить. ИМХО выстроить правильный код ревью - тот еще квест, особенно на хреновом коде
Size: a a a
2018 November 07
SK
Ну по первому я бы сказал что можно грамотно настроить Code Style, на крайний случай стянуть с GitHub гугловый code style для IDEA
По этому поводу у того же Бугаенко есть такая вот публикация (в этот раз вообще не об ООП, если что) https://cacm.acm.org/blogs/blog-cacm/229942-blame-free-quality-control/fulltext
SS
Полагаю, зависит от дотошности ревьювера. Кому то откровенно плевать на оформление, но за god object порвёт нежное место
Оформлением должен линтер заниматься, а не человек
SK
Оформлением должен линтер заниматься, а не человек
Тем не менее часто на ревью смотрят именно на кодстайл, ибо когда те прилетает ПР на 100500 строк без дескришнена, к кодстайлу прикопаться проще всего
SS
У нас таких ПРов не бывает
SK
У нас таких ПРов не бывает
Напомни - о каком стеке речь?
SS
React Native
RK
Оформлением должен линтер заниматься, а не человек
Ну знаешь, бывают случаи из разряда "Ой, ребята, а люди код ревьювят, давайте и мы", идей как правильно делать ревью нет, желание есть. Едем по код-стайлу
SS
Так в том-то и дело, что если правильно настроен линтер, то вопрос кодстайла закрыт навсегда.
SS
И это хорошо.
SK
Так в том-то и дело, что если правильно настроен линтер, то вопрос кодстайла закрыт навсегда.
В точку.
RK
Глазки в кучку, ручки из жопки ©
RK
У нас таких ПРов не бывает
Ну, хорошо вам, что сказать. Бывает и обратное.
SS
Большие PR в теории могут быть в 2 случаях: 1- большая фича, которую зачем-то делали в ветке, а не вливали каждый PR в master, 2 - большой рефакторинг, который не сделали инкрементально, что тоже ошибка
SK
Большие PR в теории могут быть в 2 случаях: 1- большая фича, которую зачем-то делали в ветке, а не вливали каждый PR в master, 2 - большой рефакторинг, который не сделали инкрементально, что тоже ошибка
Так и есть, но много ли демократов об этом задумывается?)
RK
1 мне не очень нравится, master лучше для чего-то постабильнее оставить
SS
Вливать большие изменения одной пачкой - это стабильно?
SS
Можно вместо master назвать это develop не суть
RK
master может быть веткой, скажем, monthly release?
RK
Ну да, соглашусь, не критично.