search

WePays / WePay

a web application for those who need to coordinate paying for a group.
http://wepays.herokuapp.com
MIT License
5 stars 0 forks source link

requirements.txt is incorrect... no way you require that many packages #153

Closed jbrucker closed 1 year ago

jbrucker commented 1 year ago

Looks like you generated requirements.txt using pip list or pip freeze.

I don't believe your app requires all the packages!

You should run you app in a virtualenv and determine the absolute minimum number of packages your app really needs. Start with only the ones that you know you need, such as django, python-decouple, and django-allauth.

repeat until project runs:
   delete current virtual environment    
   create a new virtualenv 
   install packages using pip -r requirements.txt
   run your project
   run your E2E test to force it to execute a lot of the code
   If it fails, look at what package is missing and add it to requirements.txt
jbrucker commented 1 year ago

Much better now. Are "wheel" and "selenium" really needed for running the app?

Tezigudo commented 1 year ago

I think it not needed for running app, Thanks for noticing us✌️