Skip to content

Deps: remove SQLite#564

Merged
mockdeep merged 1 commit into
masterfrom
rf-no_sqlite
Apr 11, 2021
Merged

Deps: remove SQLite#564
mockdeep merged 1 commit into
masterfrom
rf-no_sqlite

Conversation

@mockdeep
Copy link
Copy Markdown
Collaborator

This blocks upgrading to ActiveRecord 6.x, as the query for
true/false values is different for some reason depending on the
database, and it fails tests with SQLite. The simplest solution in this
case is probably to remove SQLite. The production setup uses Postgres,
so better to have a consistent database setup, anyway. This also frees
us up to take advantage of Postgres specific features.

@mockdeep mockdeep force-pushed the rf-no_sqlite branch 2 times, most recently from 51f47ad to 9186b6d Compare April 11, 2021 22:41
This blocks upgrading to ActiveRecord 6.x, as the query for
`true`/`false` values is different for some reason depending on the
database, and it fails tests with SQLite. The simplest solution in this
case is probably to remove SQLite. The production setup uses Postgres,
so better to have a consistent database setup, anyway. This also frees
us up to take advantage of Postgres specific features.
@mockdeep mockdeep merged commit 2e978fc into master Apr 11, 2021
@mockdeep mockdeep deleted the rf-no_sqlite branch April 11, 2021 22:46
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.

1 participant