3 ms·
> I am not versed in all the details here You should probably read the conversation in the PR: https://github.com/reactiveui/splat/pull/778 https://github.com/
by jsiepkes 5y ago
> I am not versed in all the details here
You should probably read the conversation in the PR: https://github.com/reactiveui/splat/pull/778 https://github.com/reactiveui/splat/pull/778
> there is a dire need for professional communications training among these programmers.
Maybe, but probably not why you think.
Once you've read the PR's conversation you'll see this isn't about someone just merging a PR which wasn't approved. This about someone submitting a PR, a maintainer asking for discussion before merging it, ignoring the maintainer and just merge the PR, the maintainer asking why it wasn't discussed and then making a snide remark to the maintainer.
So I agree that the "Sorry for merging a PR" isn't going to cut it here. The merging of the PR was the least of the problem. It's a hollow corporate-style apology where someone is allowed to be called out for. It's like saying: "Sorry I hurt your toe" after you pushed someone of a cliff.