3 ms·
> I’m not sure if I understand the whole idea but could you explain what this method does? Maybe...It's going to depend on the culture. If you have a passive a
by chiefalchemist 4y ago
> I’m not sure if I understand the whole idea but could you explain what this method does?
Maybe...It's going to depend on the culture. If you have a passive aggressive culture, something like this is a good fit. Otherwise, to me, you're adding friction.
Yes, it might be better to make the submitter think. But if you have to be anti-truth-seek to do it, that's a net loss.
- mytailorisrich 4y agoThis is not a question or discussion that should go in the code review record, IMO. If you do have this question during a code review then directly go have a chat with the person who wrote the code then go back to the review. It also quicker and easier.
- gtirloni 4y ago> If you have a passive aggressive culture, something like this is a good fit. That would perpetuate the passive aggressive culture.
- chiefalchemist 4y agoThat's exactly my point.