Автор статьи рассказывает, каким он был плохим на code review - унижал всех и каждого за любую мелочь, а потом понял, что так больше нельзя, и описал свои мысли по этому поводу. Если я всё правильно понял (а я всё правильно понял), то его решение - указать на пару мелких недочётов в PR, а потом самому тихой сапой поправить все косяки.
Я с этим не согласен. Это какая-то уродливая и порочная практика. Если разработчик приносит в клювике код, который ну ни в какие ворота не лезет, то пусть переделывает. Ну не в детском саду же, честное слово.
А по части того, что не стоит докапываться до совсем уж мелочей или до мест, в которых “я бы сделал иначе, но в целом норм”, я в общем-то согласен.

#codereview #code
https://habr.com/post/432822/

Название статьи попахивает буллшитом, но, как оказалось, внутри есть интересные советы. В основном мне понравились всякие штуки про поиск по коду. Я, например, ничего не знал про поиск по символам внутри файла, а фича-то полезная. Ну и Github, конечно, не IDE, но проверку небольших PR прямо там это всё немного облегчит.

#github #ide #codereview