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

Only set database_file if DATABASE_URL is missing #55

Merged
merged 1 commit into from
Sep 28, 2015

Conversation

exviva
Copy link

@exviva exviva commented Sep 24, 2015

Recently, Heroku stopped overwriting database.yml from DATABASE_URL (heroku/heroku-buildpack-ruby@be1ac0d), so, unless your database.yml contains a spec for the environment you're deploying, it fails when trying to establish the connection. We should call database= only once, either directly or from database_file=.

Recently, Heroku stopped overwriting `database.yml` from `DATABASE_URL` (heroku/heroku-buildpack-ruby@be1ac0d), so, unless your `database.yml` contains a spec for the environment you're deploying, it fails when trying to establish the connection. We should call `database=` only once, either directly or from `database_file=`.
janko added a commit that referenced this pull request Sep 28, 2015
Only set database_file if DATABASE_URL is missing
@janko janko merged commit 3631dbf into sinatra-activerecord:master Sep 28, 2015
@janko
Copy link
Member

janko commented Sep 28, 2015

Thanks! (and sorry for the wait)

janko added a commit that referenced this pull request Sep 28, 2015
Concerns #55
@exviva
Copy link
Author

exviva commented Sep 28, 2015

Thanks! ❤️

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

Successfully merging this pull request may close these issues.

2 participants