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

Already on GitHub? Sign in to your account

Run the Travis tests on MySQL, Postgres and sqlite #170

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

andyw8 commented Mar 30, 2013

No description provided.

Contributor

andyw8 commented Mar 30, 2013

Odd error from Travis, it's complaining the Gemfile is invalid, but it hasn't changed.

Contributor

mvz commented Mar 30, 2013

That probably means there's a syntax error in the database.yml file instead.

@andyw8 andyw8 referenced this pull request Mar 30, 2013

Yannick Francois Move comment queries to scopes fa14aac
Contributor

andyw8 commented Mar 31, 2013

Ok I just saw there is some custom code in the Gemfile which parses the database.yml. As the configuration uses an environment variable on Travis, I'm trying to think how this will work.

Contributor

mvz commented Apr 1, 2013

One way to solve this may be to add a special Gemfile.travis.

Contributor

andyw8 commented Apr 1, 2013

I'd be worried about that getting out of sync with the main Gemfile.

Owner

fdv commented Apr 1, 2013

We already have one for Heroku.

Le 1 avr. 2013 à 12:19, Andy Waite notifications@github.com a écrit :

I'd be worried about that getting out of sync with the main Gemfile.


Reply to this email directly or view it on GitHub.

Frédéric de Villamil / @fdevillamil
I'm not strange, weird, off, nor crazy, my reality is just different from yours.
Le Rayon UX – http://t37.net

Owner

fdv commented May 29, 2013

We now run Travis build on the 3 database engines, so I'm closing that pull request. Thank you for submitting.

@fdv fdv closed this May 29, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment