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

.travis.dist.yml: Use Travis build stages when configuring your build #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dagefoerde
Copy link
Contributor

Thanks again for having merged the --no-init option in #66. We have been successfully able to leverage it to use build stages in our projects (https://travis-ci.org/learnweb/moodle-repository_owncloud/builds/334743893, https://travis-ci.org/learnweb/moodle-mod_collaborativefolders/builds/328301352) and it's very useful.

Here's how the .travis.yml needs to be configured to make use of that option in combination with build stages.

Static checks require to be tested once, not against every possible
configuration. Therefore, introduce a single "prechecks" stage that
does all static checking, and perform dynamic testing (behat, phpunit)
against the full matrix afterwards.
@danmarsden
Copy link

just tried this out - thanks for the pointers! - FYI - I found that it didn't seem to respect the env/global settings and needed each db/moodle version to be specific in the matrix eg:
https://github.com/catalyst/moodle-tool_coursebank/blob/master/.travis.yml#L24

@Dagefoerde
Copy link
Contributor Author

Wasn't it respected at all? IIRC, if you specify variables in env/global it will be added to every element in the matrix. But it does not construct full matrices by building a cross product from different variable values, so in your case you have to specify every combination explicitly, correct. We usually do that in our plugins just like you did (https://github.com/learnweb/moodle-repository_owncloud/blob/2c0966ccde558df5e7aa9d79c5905c523e93d3de/.travis.yml#L24-L29).

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