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

Simplify Travis-CI build matrix #2655

Merged
merged 4 commits into from
Feb 15, 2017
Merged

Simplify Travis-CI build matrix #2655

merged 4 commits into from
Feb 15, 2017

Conversation

photodude
Copy link
Contributor

New simpler Travis build matrix with a few selective addons

@Ocramius
Copy link
Member

LGTM 👍

@Ocramius Ocramius self-assigned this Feb 15, 2017
@Ocramius Ocramius added this to the 2.6 milestone Feb 15, 2017
@Ocramius Ocramius merged commit 8384190 into doctrine:master Feb 15, 2017
@photodude photodude deleted the patch-5 branch February 15, 2017 10:37
@deeky666
Copy link
Member

@photodude are you sure this is still working? Having a look at this line, it does not seem that Travis is actually installing anything. The same goes for MariaDB. Also why is sudo suddenly allowed in non-sudo (docker) environments on Travis?

@photodude photodude restored the patch-5 branch February 21, 2017 16:53
@photodude
Copy link
Contributor Author

@deeky666 You are correct it's not working. This will need to be reverted as sudo doesn't work (bad suggestion from travis). I've contacted travis to see what they suggest now.

(sometimes a green light is an unfortunate appearance of a passing build)

@Ocramius Ocramius changed the title Simplify Travis Build matrix Simplify Travis-CI build matrix Jul 22, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants