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

Set up Travis CI #1

Merged
merged 3 commits into from
Apr 4, 2014
Merged

Set up Travis CI #1

merged 3 commits into from
Apr 4, 2014

Conversation

jasonmp85
Copy link
Contributor

Broke up the monolithic shell script into separate steps so the output is
clearer when it fails. These scripts are just thrown into a gist for now at
https://gist.github.com/jasonmp85/9963879. I had to enhance them to allow a
PG_PRELOAD environment variable to inform Travis what libs should be
preloaded (i.e. if the extension being tested has hooks).

jasonmp85 and others added 3 commits April 3, 2014 16:45
Broke up the monolithic shell script into separate steps so the output
is clearer when it fails. These scripts are just thrown into a gist for
now at https://gist.github.com/jasonmp85/9963879. I had to enhance them
to allow a `PG_PRELOAD` environment variable to inform Travis what libs
should be preloaded (i.e. if the extension being tested has hooks).
We want the badge to link to Travis.
@jasonmp85
Copy link
Contributor Author

In case it's unclear what this change does, it enables automatic regression tests for every commit. On the main page, I've added a badge to the README which links to the latest build and shows the current status:

screen shot 2014-04-04 at 2 14 45 am

On Pull Requests, you'll see checkboxes, crosses, warnings, and notices based on build status. This provides a quick sanity check that a pull request doesn't break existing regression tests:

screen shot 2014-04-04 at 2 14 26 am

pykello added a commit that referenced this pull request Apr 4, 2014
@pykello pykello merged commit ee9e372 into master Apr 4, 2014
@jasonmp85 jasonmp85 deleted the enable_travis branch April 4, 2014 09:08
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