Hi. I really liked Sentry since the first time I saw it some years ago. All of its features along with it being a Free and Open Source Software make it awesome!
It took to me almost two years to convince the company I work to change from Bugsnag to Sentry and one of the last problems we had with they were bad handling of Pull Requests.
Also, we maintain a big PHP codebase that has evolved thought the time and we change to Sentry to help us stay focused on the business and to care less about error handling.
So, it was a surprise to me opening a PR requesting the change of only one character, in order to make a deprecated and unused value to be marked as nullable, and see a rude and tough response. It seemed to me that it was all about a discussing between personal values (disguised as good practices) vs a pragmatical need. Along with some people that I didn’t even know if they could help, as they are not in the Sentry’s GitHub organization.
I know some nice people that have developed Sentry and I can’t imagine them making this fuzz because of a so simple change while creating falsifiable arguments that also can be simply fixed, like passing through a null value (which could be handled with a few more characters and no side-effects) or which part of a short argument is wrong.
Well, by the end I want to leave this quote that says the main concern about it all: “People may forget what you say. They may forget what you did, but people will never forget how you made them feel.”