search

UkraineNow-Intel / autoSA-backend

Django backend for autoSA
0 stars 1 forks source link

Rework requirements.txt in backend repo to only list necessary reqs #19

Closed j-bennet closed 2 years ago

j-bennet commented 2 years ago

It looks like our requirements.txt was created by someone who simply piped pip freeze output into the file.

This is not a good practice. This approach dumps all installed packages into the list of requirements, including:

  1. dependencies of packages
  2. whatever you installed for debugging, example: ipython or ipdb
  3. whatever package you installed to play with, but ended up not using

Dependencies of packages may chain. And if we're pinning them exactly, we'll break the dependency. The other two don't need explaining.