618 karma · joined October 7, 2019
“Don’t be condescending” seems like generally applicable advice. But IMO, sometimes you just know something the code submitter doesn’t (or vise versa) and discussing that can be useful. And i think that’s pretty much teaching!
If comments don’t impact the code base or the people involved, then yeah they’re not useful.
I get this but I prefer the opposite. By over communicating your preferences, asking questions, leaving “if I follow this correctly” comments, and generally not holding back, you create lots of chances for discussion and knowledge sharing. And you’ll catch more bugs.
This is especially true when a junior dev is involved.
My only exposure to grammarly is YouTube ads, and I noticed they often show women struggling to write (I think 3/4 of their actors are female). Two theories:
- grammarly is subtly sexist
- most characters in ads are women
Anyway I never had an outlet for this thought so posting here
(Also to be fair they’ve sold a handful of cars…)