this post was submitted on 03 May 2026
17 points (90.5% liked)

Web Development

5626 readers
74 users here now

Welcome to the web development community! This is a place to post, discuss, get help about, etc. anything related to web development

What is web development?

Web development is the process of creating websites or web applications

Rules/Guidelines

Related Communities

Wormhole

Some webdev blogsNot sure what to post in here? Want some web development related things to read?

Heres a couple blogs that have web development related content

CreditsIcon base by Delapouite under CC BY 3.0 with modifications to add a gradient

founded 2 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
[โ€“] jtrek@startrek.website 4 points 15 hours ago (1 children)

This doesn't spend much time on the scenario where it's not two peers in the review, but like a junior dev and senior dev.

I recently did a code review for someone who is very junior ^1^ , and after spending like two hours fixing all the syntax errors and low level goblins, I realized I couldn't see the forest at all. I'd been staring at the "this variable is undefined in this case" trees so much. And then they were getting antsy and management was getting antsy...

^1^ They're not actually junior. They've been in this role for years. Unfortunately, this org has like no mentorship and no standards, so they haven't actually learned much in all that time. They've just been copy-pasting code until stuff works.

after spending like two hours fixing all the syntax errors

If the code has syntax errors then clearly the tests don't pass. A minimum requirement to start reviewing a PR should be that the tests all pass.