ZoomyPotato
ZoomyPotato

Comments on PRs

Are getting comments on PRs a grave in IT?

10d ago
Talking product sense with Ridhi
9 min AI interview5 questions
Round 1 by Grapevine
ZestyPotato
ZestyPotato

Nope. Best way to learn is when someone comment on your PR 🙂‍↕️

ZoomyPotato
ZoomyPotato

But they are using it against me saying I am delivering low quality code

ZippyPickle
ZippyPickle
Student10d

Your code != You

Always good to have a second set of eyes going through the changes

FluffyDonut
FluffyDonut
IBM10d

Not really, You are lucky if someone is going through your PR thoroughly and giving feedback. Sometime those feedback are just personal opinions (for example instead of writing null == value you should write value == null). So there’s no harm in them, In fact reflecting on those comments help you become better with your code you will understand handling edge cases, you will get to know sometimes additional code might not be needed for something. Overall comments aren’t a bad thing. Just make sure to reduce the number of comments over time by following good practices.

Discover more
Curated from across
Software Engineers
by ZippyCupcakeStudent

Am I overreacting? How to handle PR comments

Have this co worker, who kind of leads my current project. He keeps digging into PRs and adding comments even when others from the team approved it.

How do I politely shut down this activity? or do I have to not take offense and just wo...