logoalt Hacker News

giancarlostorotoday at 3:19 PM2 repliesview on HN

The other issue with code review, and I'm glad I've not worked with people like this anymore, for the person being reviewed: NOBODY IS ATTACKING YOU, nobody is saying your code is bad, the goal is to do a once-over for quality.

Another goal people often miss:

It's okay to ask "stupid questions" and I would argue as a Junior, ask away, even if no code changes happen, ASK. Kind of follows the spirit of the original post, which is, can you maintain this?


Replies

oh_my_goodnesstoday at 3:21 PM

>NOBODY IS ATTACKING YOU

Somehow this captures a lot of the culture for me.

show 1 reply
threethirtytwotoday at 4:26 PM

Not true. I’ve attacked people before in the sense I was disgusted with the short cuts they took and the things they tried to get away with in the code.

I’ve seen the malicious deletion of features in order to stay hidden and I’ve called it out.

show 1 reply