Hacker News new | ask | show | jobs
by ffsm8 703 days ago
That's precisely why I love to only comment things with questions, i.e. "isn't go-to considered harmful? Is this really the best way?".

It gives the person making the change the opportunity to give context for their decision - and either change or spell out their reasoning for it, potentially giving me an opportunity to learn from them.

Another good benefit is that it doesn't "attack" the PR author in the "non-violent communication" way

1 comments

Doesn't that seem a bit passive aggressive? See what I did there?
Surely it depends on both your phrasing and how you usually interact with your colleagues, no?
You can totally do this with non-passive-aggressive phrasings ("Is this possible using structured programming instead of GOTO? What went wrong when you tried, how messy does it get?")