For further actions, you may consider blocking this person and/or reporting abuse
Read next
Populating a Vertica Database with Random Data
Dmitry Romanoff -
Fixing VS Code Go to Definition Issue on Linux Mint
Mark Adel -
Liquidation is every crypto trader's worst nightmare. But what can be done to avoid it?
Alex Roor -
I Built a CLI Tool That Tells Me When I'm Over-Engineering (Again) 🤖
Pruthvi Kumar -
Top comments (4)
Constructive code reviews. Nothing kills the motivation of team members faster than code reviews that either don't happen (resulting in the change requests atrophying) or do happen and they're either overly critical or held up due to minor or cosmetic improvements.
Less criticism means things can change faster, to avoid code being obsoleted in review. But we also have to be careful of too little criticism and lgtm culture.
I completely agree, I'm not sayjng we don't highlight issues at all. As a rule, I tend to highlight both positives and negatives in a review as well as being explicit when a cosmetic or minor change should not block an otherwise acceptable change request. These changes should not block a PR and can be fixed in a follow-up change.
Ah yes, very elusive in our field