-
Notifications
You must be signed in to change notification settings - Fork 320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The firebase-admin package contributes 151.7 MB of dependencies #803
Comments
Hello @lahirumaramba, |
Hey @VishalGawade1 we always appreciate contributions to the code base! Feel free to take a stab at it if you would like to contribute to the repo. You can find the guidelines on https://github.com/firebase/firebase-admin-python/blob/master/CONTRIBUTING.md |
cc: @jonathanedey |
Thanks a ton! Let me see if I can do anything :) |
Hello @lahirumaramba , My plan is to make some of the larger dependencies optional using "extras_require" in setup.py. Specifically, I’d move dependencies like google-cloud-firestore and google-cloud-storage to optional extras. This way, users can install just what they need (e.g., pip install firebase-admin[firestore]), which should cut down the default install size quite a bit. The change would include: Making Firestore and Storage optional dependencies |
Hi @VishalGawade1, |
Hi @jonathanedey I’ll also update the docs to make this clear. Let me know if this works! |
[REQUIRED] Step 2: Describe your environment
[REQUIRED] Step 3: Describe the problem
We recently performed an audit of our dependencies and firebase-admin is by far larger than anything else, clocking in at around 151.7 MB, including its transitive dependencies, according to the measuring approach we used (see below).
All we want is the Cloud Messaging part but I guess we're getting the kitchen sink and the house as well. ;)
Steps to reproduce:
Install just firebase-admin in a fresh virtualenv and look at the size of installed artefacts. We used the python-package-size tool to measure this but it can be done manually.
Relevant Code:
N/A
The text was updated successfully, but these errors were encountered: