Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to using dotenv instead of yaml configuration files #5

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

nuclearsandwich
Copy link
Member

YAML config files are difficult to maintain and grow and present challenges with managing configuration through local development and production environments.

According to the 12 Factor Application, configuration belongs in the Environment rather than in settings files. Heroku uses environment variables as the Application-facing interface to the heroku config command.

This PR will start us out using a gem called dotenv to load environment variables needed in development from a file called .env in the application directory.

This file will be ignored by Git so that each individual contributor's specific development configuration is not overwritten by changes made by others.

To start off new developers a bootstrapping script will be created which sets up a local development environment in as few manual steps as possible.

Ready status 🚥

  • Include dotenv-rails gem
  • Add .env to .gitignore
  • Write script/bootstrap program
  • Update README with new bootstrap instructions
  • Test on Mac OS X / Linux
  • Test on Windows
  • Remove config/database.yml from repository.

@nuclearsandwich
Copy link
Member Author

@iTweak0r do you still do Rails development on Windows? I might stop trying to make this cross platform right away and work on it later.

@nuclearsandwich
Copy link
Member Author

@marcyDel marcyDel closed this Jan 31, 2014
@marcyDel marcyDel reopened this Jan 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants