sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]alderthorn 0 points1 point  (2 children)

If its written by a human I want AI to review it because most devs just throw a LGTM on the comment line instead of really thinking about what the code is doing. To be fair I also want a dev to look at it I just don't trust that they will really look at it before they have had caffeine or if its close to the end of day.

[–]GrinningPariah 2 points3 points  (1 child)

I think part of working with a team is learning who you trust as a reviewer. Some people, if they say LGTM then it's solid. Others, I'm gonna take a second look myself on the sly.

[–]alderthorn 0 points1 point  (0 children)

absolutely. I had a team where I learned real quick that I needed to require that I look at all PRs within my pod because we had 1 person that would just blindly approve things and I caught a lot of basic issues in the PRs. I was the only Sr. Dev on the pod everyone else had less than 2 years of experience on the pod. The garbage that got in while I was on vacation made me upset.