News for hackers. Stuff that matters.
Re:No code reviews by default by kps on Tuesday January 04, 2022 @04:22PM (#29800718)
> Only for a drive-by take-down by someone with none of the context.

Then write it down. If the code reviewer can't follow what's going on, what hope is there for the new hire looking at it six months from now?

Re:No code reviews by default by ipaddr on Tuesday January 04, 2022 @04:45PM (#29801006)
Because looking through 6 month old prs are what new hires are doing to learn the codebase?
Re:No code reviews by default by fragmede on Tuesday January 04, 2022 @05:44PM (#29801875)
Running through `git blame` and looking at the commit message, the PR, and the PR comments is a very practical way to learn about a codebase! Beyond some high level code organization stuff that exists in a readme, learning a codebase is really about getting a history lesson of how the product evolved, the company pivoted, and the team re-org'd.

It'll be 6-months if you're lucky. Try figuring out why there's a particular "if" clause, one or two or four years later. Software maintenance, especially when the original author has moved onto another team/organization/company/career, is a frustrating art of almost remembered stories and hoping that you can figure out Chesterton's fence, lest it become Chesterton's barbed wire. The worst is when you fix a small bug with code and introduce an even bigger big in the process.

Re:No code reviews by default by detaro on Tuesday January 04, 2022 @05:37PM (#29801760)
Not necessarily looking through PRs, but presumably they need to work with actual code in your code base that was merged at some point, unless your product is growing so fast that new people just write new code all the time? (Same applies for not-new coworkers that now need to touch that code area anyways)
Re:No code reviews by default by yxhuvud on Tuesday January 04, 2022 @05:36PM (#29801744)
Sure. If they are good and have a decent grasp of using version control then will look at the history of a certain piece of code if they don't understand why it is the way it is. I do it often, even when I'm no longer a new hire.
Re:No code reviews by default by lalaithion on Tuesday January 04, 2022 @05:35PM (#29801727)
That is one of the things I do to learn new codebases.