News for hackers. Stuff that matters.
Re:There’s No Such Thing as Clean Code by skohan on Friday January 28, 2022 @05:24AM (#30112857)
My biggest problem with code reviews is that they are so often without value. Code review should be a chance for teams to learn from each other, catch issues, and align on coding practices and strategies. In my experience, a lot of the time code reviews are either just a task - where people try to find superficial "mistakes" just to tick a box, or worse a chance for certain colleagues to power trip and try to get other team members to submit for it's own sake.
Re:There’s No Such Thing as Clean Code by zoomablemind on Friday January 28, 2022 @07:45AM (#30113882)
> ... Code review should be a chance for teams to learn from each other, catch issues, and align on coding practices and strategies.

IMO a practical benefit of a routine code review is just a shared understanding. As simple as that. Does the code look to be doing what it says it should do? Do the tests express the intended behavior?

Of course, the style, the quality of code could be of attention too, but that's extras, in my opinion, as long as the functionality is understood and confirmed.

I like relating this to an effort of understanding someone who speaks with an accent, be it a local dialect or foreign-originated. It does require some tolerance, but the shared understanding is the goal.

Re:There’s No Such Thing as Clean Code by muttled on Friday January 28, 2022 @11:17AM (#30116601)
> where people try to find superficial "mistakes" just to tick a box

I've found this to be true in change control as well as any meeting involving people who feel they are compelled to "contribute" even if they don't have anything of value. You don't want to seem like the person who's just attending the meeting and slacking off, so if you're expected to contribute and you can't find any useful critiques, you just pick nits.