Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Setup Travis CI #65

Merged
merged 1 commit into from
Sep 30, 2016
Merged

Setup Travis CI #65

merged 1 commit into from
Sep 30, 2016

Conversation

makenova
Copy link

use virtual screen for browser tests
run tests once and use chromium instead of chrome for Travis CI

@makenova
Copy link
Author

There were some errors in the CI with the submoudules so I told Travis to ignore it.

Also, Chrome isn't available in the container where the tests are run so we have to use Chromium.

If you're interested, I found the details for the chromium fix here, via this issue.

@smurphy8
Copy link

Awesome, is there a link to the CI in action somewhere?

@makenova
Copy link
Author

Yup. Right here.

I can also add the badge to the Readme file.

@smurphy8
Copy link

That would be cool, thanks!

On Sep 30, 2016 08:48, "Makenova" [email protected] wrote:

Yup. Right here https://travis-ci.org/codeforokc/openbudgetokc.

I can also add the badge to the Readme file.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#65 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALJjHK6AZqXlqzBL_6SaNG6lMyw1Bb7ks5qvRNJgaJpZM4KKtMO
.

use virtual screen for browser tests
run tests once and use chromium instead of chrome for Travis CI
Add Travis CI status image/badge
@makenova
Copy link
Author

There isn't actually a build for the master branch yet, so I think the badge is going to remain status unknown until this branch is merged.

@smurphy8
Copy link

K

On Sep 30, 2016 09:17, "Makenova" [email protected] wrote:

There isn't actually a build for the master branch yet, so I think the
badge is going to remain status unknown until this branch is merged.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#65 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALJjEyCc_Opbf9P8l3xYaj9n1lFK5xJks5qvRnagaJpZM4KKtMO
.

Copy link
Member

@mkchandler mkchandler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all looks great!

@mkchandler mkchandler merged commit 5173c62 into codeforokc:master Sep 30, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants