this post was submitted on 28 May 2024
118 points (96.1% liked)

Programming

17450 readers
78 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev



founded 1 year ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
[โ€“] NotMyOldRedditName@lemmy.world 4 points 5 months ago* (last edited 5 months ago) (1 children)

This is my typical experience as well, too many people don't do a code review of their own PR first.

When I was a junior, I had this coworker who did all my reviews. I was doing my absolute best and wanted to show that I was learning, so I would review all my work before submitting it and think, how would he review and respond to this code.

That just stuck with me and it's my normal practice now.

I eventually learned that's not as normal as I thought. I also tend to give better code reviews than others.

Edit: the other thing I do is check in with who will be reviewing my code well before I submit anything someone might think is weird and have a discussion about it before the reveiw. If it's weird, there might be a better way unless were stuck due to technical debt or something, and doing that early vs at the end usually saves time.

[โ€“] Fades@lemmy.world 2 points 5 months ago

I had the exact same path to that habit as well! Good mentors make good devs!!