Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

Load application configuration from file #159

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tsmolka
Copy link

@tsmolka tsmolka commented Jul 29, 2016

Hi, what about moving settings to an external file? It's a good practice and simplifies configuration. Other settings (e.g. db file) should ideally be moved there as well.

@sroberts
Copy link
Contributor

sroberts commented Aug 1, 2016

So that goes pretty aggressively against the 12 Factor App idea of keeping configuration out of code.

@k3vb0t
Copy link
Contributor

k3vb0t commented Aug 9, 2016

Just to chime into this, with respect to the 12 Factor App rules, I use configparser and usually make a config.ini( or something there of) that works really well imo. As long as .gitignore has that file in it, you should be good to go. TBH, I hate having to remove and re-add my API keys to the file I'm pushing/pulling each time.

To note, I'm not even close to being a developer, but as someone who does numerous pushes and pulls, I like the notion. Just my 1 cent.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants