It's time to stop doing code reviews
blog.brokk.ai
It's time to stop doing code reviews
1–10 of 10 posts
Re: It's time to stop doing code reviews
#2Because there are better ways now to solve the problems that code review exists to solve.
Re: It's time to stop doing code reviews
#3I wonder what's going to happen after applying no-review practice, when nobody knows what code-base actually does, in enough details to provide technical guidance?
Re: It's time to stop doing code reviews
#4How are you going to know the AI did not build the wrong thing? Not entirely wrong, but they will make some bad decision without consulting you. Even with a spec, AIs find a way to do it. (Humans can do this too.)
Re: It's time to stop doing code reviews
#5> If I were looking for work as an IC, I would avoid orgs that still practice this as a kind of make-work nearly as offensive as gratuitous meetings.
Would you get on a plane knowing its systems weren't code reviewed by humans?
Re: It's time to stop doing code reviews
#6> Technical debt is nearly a nothingburger now. If you went in the wrong direction, just... change the code to go in the right direction.
Uncharitable though it may sound, what I hear reading this is "Tell me you've never worked on a large system without telling me you've never worked on a large system."
Re: It's time to stop doing code reviews
#7> If I were looking for work as an IC, I would avoid orgs that still practice this as a kind of make-work nearly as offensive as gratuitous meetings. Would you get on a plane knowing its systems weren't code reviewed by humans?
No. Not only that, I would not get on a plane where a defined and followed process was not used in the development of the software.
Code review is one step of many.
Re: It's time to stop doing code reviews
#8I wonder what's going to happen after applying no-review practice, when nobody knows what code-base actually does, in enough details to provide technical guidance?
Consultants will make a mint dealing with these systems. It will make Y2K look like a nothing-burger.
Re: It's time to stop doing code reviews
#9I know reviewing code on github sucks, which was my source of frustration when reviewing code at work.
Please use pyor.review, it will make reviewing PRs a heck of a lot easier (i built it myself)
even though I use the latest and greatest models, I still have to review the code AI generates, and it still makes mistakes, I still find it making weird decisions, slop blocks of code, unnecessarily long solutions that can be replaced with a couple lines of code.
code review is required to build good software, and that what separates engineers from vibe coders. you're missing the "engineer" part from the role, and that's still here to stay.
Re: It's time to stop doing code reviews
#10I agree with the direction, but I think "stop doing code reviews" is slightly too far. The review boundary should move up. Engineers should review the plan made by agents and trust they will carry out the implementation correctly.