257m@lemmy.ml to Programming · 1 year agoWhat are your programming hot takes?message-squaremessage-square897fedilinkarrow-up1353arrow-down114
arrow-up1339arrow-down1message-squareWhat are your programming hot takes?257m@lemmy.ml to Programming · 1 year agomessage-square897fedilink
minus-squareStudioLElinkfedilinkarrow-up14·1 year agoMy mantra has always been to bring solutions not problems. Applying that to code reviews makes for a far more productive experience. Rather than just pointing out errors in code help the developer with prompts towards the solution. Or, if you’re too lazy to explain why something shouldn’t be done then why should another developer have to act on your criticism?
minus-squaremuddi [he/him]@hexbear.netlinkfedilinkEnglisharrow-up3·1 year agoSame , but in my experience this can backfire even worse because the other person thinks you are trying to show off as well as make them look bad. But the problem is less with you and me, and more with this type of person
My mantra has always been to bring solutions not problems. Applying that to code reviews makes for a far more productive experience.
Rather than just pointing out errors in code help the developer with prompts towards the solution.
Or, if you’re too lazy to explain why something shouldn’t be done then why should another developer have to act on your criticism?
Same , but in my experience this can backfire even worse because the other person thinks you are trying to show off as well as make them look bad. But the problem is less with you and me, and more with this type of person