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

Reinstate OSX on Travis #798

Merged
merged 2 commits into from Apr 19, 2017

Conversation

Projects
None yet
2 participants
@patricksnape
Contributor

patricksnape commented Apr 19, 2017

No description provided.

@patricksnape patricksnape changed the title from Try adding OSX to Travis to Reinstate OSX on Travis Apr 19, 2017

@patricksnape

This comment has been minimized.

Contributor

patricksnape commented Apr 19, 2017

@jabooth Happy?

.travis.yml Outdated

git:
depth: 200
depth: 1

This comment has been minimized.

@jabooth

jabooth Apr 19, 2017

Member

Why change this, didn't we have issues with insufficient depth in past?

This comment has been minimized.

@patricksnape

patricksnape Apr 19, 2017

Contributor

I don't believe so - was just cleaning up the YAML a bit

.travis.yml Outdated
@@ -22,14 +30,4 @@ script:
- ~/miniconda/bin/python condaci.py build ./conda

notifications:
slack:

This comment has been minimized.

@jabooth

jabooth Apr 19, 2017

Member

Any reason we are disabled slack build notifications too (or is this how handled though a proper integration? Don't remember)

@patricksnape patricksnape force-pushed the patricksnape:travis_osx branch from c511e1c to 261b9c6 Apr 19, 2017

@patricksnape

This comment has been minimized.

Contributor

patricksnape commented Apr 19, 2017

OK I went back to minimal changes

@jabooth

This comment has been minimized.

Member

jabooth commented Apr 19, 2017

Cool, less risk of issues. I'm happy merge whenever you're happy CI is passing

@patricksnape patricksnape merged commit 1bdd857 into menpo:master Apr 19, 2017

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@patricksnape patricksnape deleted the patricksnape:travis_osx branch Apr 19, 2017

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