10 sats \ 7 replies \ @nemo 20 Aug 2023 \ parent \ on: Code review tool recommendations tech
deleted by author
In some cases the code is fine but I suspect it could have uncaught corner cases or improved but I'm not 100% sure and I don't want to "alert" the author if I'm wrong.
reply
Corner cases are were code reviews are really helpful! It's taken many years to separate "you should have done it how I would have done it" from "this code is ok to be merged". It's helpful to think "does this pr improve the code overall".
Bugs will happen. Keeping the critical sats-related code separate from the rest, + CODEOWNERS will keep the critical sections safer while letting the rest change more quickly.
reply
deleted by author
reply
deleted by author
reply
deleted by author
reply
Noooo I don't want more tools lol. I want one really good one. 😄
reply
deleted by author
reply