Remove configs.py

[haneslinger]

We dont need this anymore, we've got .env

[MatthewSteen]

LGTM, just one suggestion and needs a test fix.

[TShapinsky]

Looks good, we could also consider incorporating or linking to python-dotenv which automatically brings in a .env file as environment variables https://pypi.org/project/python-dotenv/

[TShapinsky]

Looks good to me once tests pass and it's merged

[TShapinsky]

1. Provide a couple of examples for loading env variables on windows/linux/mac
2. Link to python-dotenv as a possible way to load from .env cleanly

[haneslinger]

@MatthewSteen re-pinging this.

[MatthewSteen]

@haneslinger I think this just needs some brief documentation about the BASH requirement to run on Windows. See my comments and suggestions above, specifically #256 (comment).

Looking at this a bit more, I think it just requires a Windows user to use bash, which is included with Git. So maybe just keep as is and note that in the docs, which we could revisit later. The python-dotenv solution could be clean but I generally don't like adding dependencies if they can be avoided.

[haneslinger]

@MatthewSteen Oh, yes, thank you. Would you be able to add that? Do you want it in this PR?

[MatthewSteen]

@haneslinger I added a note for windows users, which I think is sufficient for now. Take a look and merge when ready.