Someone want a modification to be merged and push a lot of template code giving a 3 line PR explanation.
This kind of situation comes often in OSS, specially in C++ project and I wouldn't be surprised if OMW had a lot of PR like this.
With this kind of PR, the maintainer have to go into each line and try to understand what the point because the original author didn't explain that much + it's C++ template and not everyone like that.
This is not a excuse to be arsh or anything, another option would have been to put a tag "need clarification" and let the PR in a hole and force the author to motivates its modifications.
Accepting failure is integral to learning and growth, but a key part of that is identifying the weak points and learning from them.
Edit: s/gift wrap/sugar coat
> Now I am rereading my first comment an see that it is too rude. I am sorry for that. I was in a bad mood when wrote this.
As a native English speaker: no it doesn't. Whoever told you that was gaslighting you, a corporate-politically-correct shitheel, or both. (Or being sarcastic, or cargo-culting something they picked from one of the previous categories, or some other breakdown of communication.)
> People
I'm a bit confused, are you talking about people and countries or not? In your original comment you explicitly said "people" and "countries"
https://github.com/OpenMW/openmw/pull/3176#discussion_r72858...