Comment

Comment by bobthepanda

YMMV but i work at a place where

* code review is expected responsibility, so everyone participates in every part of it regularly, so they are also incentivized to keep the process sane

* we have an auto linter and we recommend saving on fix specifically so no one argues about useless style nits

* CR back and forth is measured in minutes or hours so you are not waiting days to resolve someone’s drive by comment

* CR feedback always has a specific action item that is easy to address

* reviewees submit smaller CRs which are quick and easy to review for reviewers

parent bobthepanda 3 years ago

Discussion

Live from the official Hacker News API

A modern reader for Hacker News · SvelteKit + Skeleton