search

Direct4Ag / ag-services

1 stars 0 forks source link

updated structure #12

Closed diegoac2 closed 2 years ago

diegoac2 commented 2 years ago

updated folder structure (moved everything under app folder)

navarroc commented 2 years ago

Minor nitpick, but maybe consider putting "config.py" in a sub-folder of app, maybe app/core to keep the top level cleaner. Later we can put a folder called "tests" inside app (e.g. app/tests).

navarroc commented 2 years ago

I think the Dockerfile needs to be updated for the package structure change. I think the CMD line needs to be:

CMD gunicorn -b 0.0.0.0:8000 -w ${WORKERS} -k uvicorn.workers.UvicornWorker app.main:app