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.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
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.