Skip to content

Add node/PG version matrix to Travis-CI config #761

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

Merged
merged 5 commits into from
Aug 4, 2015

Conversation

sehrope
Copy link
Contributor

@sehrope sehrope commented Apr 4, 2015

This PR updates the .travis.yml config file to test against various permutations of node and PostgreSQL versions. I've also removed an unused shell script for setting up PG 9.2. It wasn't referenced anywhere and Travis takes care of setting up the PG environments via the matrix config.

Previously the Travis config was only testing against node v0.10 and v0.11.

With this PR it now tests against node versions v0.10, v0.11, v0.12, and io.js v1, and PostgreSQL versions 9.1, 9.2, 9.3, and 9.4 for a total of 4 x 4 = 16 test environments (plus 1 more for whatever the defaults are).

@sehrope
Copy link
Contributor Author

sehrope commented Apr 4, 2015

Looks like there is some kind of transient error running the tests for v0.11/9.3 on Travis: https://travis-ci.org/brianc/node-postgres/jobs/57163323#L206

I suggest running that build again as it worked fine when I ran it (as well as the 16 other env combinations).

sehrope added 5 commits August 2, 2015 11:03
Add a matrix configuration to .travis.yml to test permutations of node and PostgreSQL.
Node versions tested against are v0.10, v0.11, v0.12 and io.js v1.
PostgreSQL versions tested against are 9.1, 9.2, 9.3, and 9.4.
This enables container based builds on Travis which should speed
up automated testing.
@sehrope
Copy link
Contributor Author

sehrope commented Aug 2, 2015

I've updated this PR to remove node v0.11, bump the iojs version to iojs-v2 and switch to container based builds in Travis (by disabling sudo in .travis.yml).

The switch to container based testing speeds up the automated tests quite a bit. It's enough that running the full matrix with multiple node versions (0.10, 0.12, and iojs-v2) and PostgreSQL server versions (9.1 through 9.4) takes the same amount of time as the existing tests with the non-container setup.

@brianc
Copy link
Owner

brianc commented Aug 4, 2015

This is amazing! Thank you so much!!

brianc added a commit that referenced this pull request Aug 4, 2015
Add node/PG version matrix to Travis-CI config
@brianc brianc merged commit 715e500 into brianc:master Aug 4, 2015
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