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

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.

3 participants