You copied that function without understanding why it does what it does, and as a result your code is flawed & inefficient. This poor practice is a pattern I’ve noticed.
Less likely to be effective. There’s a good chance that the submitter won’t get the message, and that they’ll submit another pull request, five minutes later, with the exact same issue that made the first PR to be rejected. And again. Again. Again.
More insulting. Now you aren’t just saying “your code is garbage”; you’re saying “your code is garbage and you’re a fragile little thing that will break apart if handled incorrectly”.
As likely to create drama as the original verbiage, given that the drama is originated in human nature - we humans want to believe (even if outright false) that we’re “contributing”, even when we are not.
What’s your opinion on this alternative verbiage?
My opinion is that it is:
It will fall on deaf ears