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

Properly support DATABASE_URL #112

Merged
merged 1 commit into from
Apr 14, 2015
Merged

Properly support DATABASE_URL #112

merged 1 commit into from
Apr 14, 2015

Conversation

MattFenelon
Copy link
Contributor

Switched out the code for loading configuration for Rails's version of
loading database configuration.

Rails has implemented support for DATABASE_URL in rails/rails#13463 and
has dealt with the possible combinations of having DATABASE_URL set and
a YAML config file, see rails/rails#13463 (comment).

There is a breaking change in the way the hash is loaded; Rails does not
convert the keys of the hash to symbols, instead they are left as
strings; any code reliant on the hash keys being symbols will break.

Switched out the code for loading configuration for Rails's version of
loading database configuration.

Rails has implemented support for DATABASE_URL in rails/rails@13463 and
has dealt with the possible combinations of having DATABASE_URL set and
a YAML config file, see rails/rails#13463 (comment).

There is a breaking change in the way the hash is loaded; Rails does not
convert the keys of the hash to symbols, instead they are left as
strings; any code reliant on the hash keys being symbols will break.
@MattFenelon
Copy link
Contributor Author

This fixes #111.

twobitlabs added a commit that referenced this pull request Apr 14, 2015
Properly support DATABASE_URL
@twobitlabs twobitlabs merged commit 2dff75e into thuss:master Apr 14, 2015
@twobitlabs
Copy link
Collaborator

Thanks for the pull request, published 4.0.2 of the gem with this included.

@MattFenelon
Copy link
Contributor Author

thanks @twobitlabs

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