Reduce Dependencies #1515
Replies: 2 comments
-
First and foremost:
I'm sorry, but this is a big understatement. Please read through all the code that it's in here before expressing something like this. This packages is now so slim because it went through a lot of work (disclaimer: mine too) and also exactly because it leverages those dependencies. Ignoring this, and going over the current dependencies, these are the current ones: Lines 23 to 37 in 9e53a01
Basically, we can decide to drop only two of them. reducing everything else by 3 tops. The core of the matter is the HTTP client, and that's a must. If you're having conflicts of any kind, we may investigate those, but I fear it's just out-of-date packages, and you can overcome that by installing older versions of the SDK. |
Beta Was this translation helpful? Give feedback.
-
Hey @abhibeckert Over the last couple of months, we have already eliminated some dependencies. We will re-evaluate our dependencies in an upcoming v4, but at the moment, I don't see much we could |
Beta Was this translation helpful? Give feedback.
-
Problem Statement
I'd really like it if this package had fewer dependencies - composer installed 21 packages when I installed Sentry just now (it has more than 21 dependencies, but those are the ones I didn't already have).
It took me a couple of hours to figure out what changes I needed to make to my composer file to be able to install Sentry at all. Some of your dependencies had dependencies with dependencies that didn't match the version numbers required by dependencies of some of my dependencies... next I need to work out if the changes I made are going to cause problems elsewhere...
Solution Brainstorm
I'd feel a lot more comfortable using Sentry if it had less dependencies. Preferably none. I feel like this is achievable considering what this package does is relatively simple.
Beta Was this translation helpful? Give feedback.
All reactions