I always try to review the code anyway. There's no guarantee that what they wrote is doing what you want it to do. Sometimes I find the person was told to do something and didn't realize it actually needs to do Y and not just X, or visa versa.

[–] [email protected] 20 points 5 months ago (1 children)

I like to shoot for the middle ground: skim for key functions and check those, run code locally to see if it does roughly what I think it should do and if it does merge it into dev and see what breaks.

Small PRs get nitpicked to death since they're almost certainly around more important code

[–] derpgon 10 points 5 months ago

Especially when you see a change in code, but not in tests ☠️

[–] [email protected] 2 points 5 months ago (1 children)

Yes, I always review the code, just avoid nitpicking the hell out of it.

[–] [email protected] 1 points 5 months ago

Yeah, sorry, totally misread your comment.