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

update travis config #16

Merged
merged 4 commits into from
Nov 19, 2014
Merged

update travis config #16

merged 4 commits into from
Nov 19, 2014

Conversation

gurneyalex
Copy link
Member

use UNIT_TEST=1 so as not to have to worry about installed modules breaking the
tests of other modules

use UNIT_TEST=1 so as not to have to worry about installed modules breaking the
tests of other modules
@guewen
Copy link
Member

guewen commented Nov 4, 2014

👍

1 similar comment
@yvaucher
Copy link
Member

yvaucher commented Nov 4, 2014

👍

@gurneyalex
Copy link
Member Author

@guewen @yvaucher you approved a bit too soon...

@guewen
Copy link
Member

guewen commented Nov 4, 2014

I approved the principle ;-) But yes, I could have retain me and wait for the travis' result.

@guewen
Copy link
Member

guewen commented Nov 4, 2014

Is it due to missing dependencies?

@yvaucher
Copy link
Member

yvaucher commented Nov 4, 2014

💥

erm this repo is not ready to use this

@coveralls
Copy link

Coverage Status

Coverage increased (+1.33%) when pulling 57d26a1 on gurneyalex:8.0-update-travis into e55651c on OCA:8.0.

@yvaucher
Copy link
Member

yvaucher commented Nov 6, 2014

Restarted travis

@coveralls
Copy link

Coverage Status

Coverage increased (+2.63%) when pulling 2bb1b63 on gurneyalex:8.0-update-travis into e55651c on OCA:8.0.

@pedrobaeza and @dreipt argued this was a good idea
@coveralls
Copy link

Coverage Status

Coverage increased (+1.76%) when pulling b7c3b6a on gurneyalex:8.0-update-travis into e55651c on OCA:8.0.

@nbessi
Copy link

nbessi commented Nov 19, 2014

👍

nbessi added a commit that referenced this pull request Nov 19, 2014
@nbessi nbessi merged commit 325ee2f into OCA:8.0 Nov 19, 2014
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.

None yet

5 participants