Does this mean we should remove the if os.getenv("PORT") else 8080 from the main.py?

Does this mean we should remove the if os.getenv("PORT") else 8080 from the main.py?

No, this is only for if you want to run the docker image locally. Without the PORT variable set, the container won't start! This is because of the line in the Dockerfiles: CMD exec gunicorn --bind :$PORT --workers 1 --threads 8 main:app

You can still run the python file outside of a docker image, in which case, you need the getenv, eg: python main.py

Once this passes the appropriate CI, I'll merge it in! I know that this uses f strings and we previously said that was okay for it to fail earlier python versions. :)