Skip to content
This repository has been archived by the owner on Apr 23, 2021. It is now read-only.

setup a stable and latest build in travis ci #19

Closed
wants to merge 10 commits into from

Conversation

itziakos
Copy link
Member

@itziakos itziakos commented Apr 2, 2015

This PR fixes #17. We setup two builds in travis-ci one working on the latest stable release of simphony-common that openfoam master supports and the other on the latest master version of simphony-common. The cutting edge build failure or success will not affect the commit status on PRs but will provide a preview on what currently breaks and will need to be fixed in order to use the latest simphony-common code.

@itziakos
Copy link
Member Author

@khiltunen, This PR is ready for review.

We setups two builds on travis-ci for every change that is pushed to github. The first build is against a stable version of simphony-common (currently 0.1.2) and the second build is for then latest master commit of simphony-common. The seconds build is designated as a build that can fail

This setup will help when you are adding features in the openfoam-simphony wrapper. By selecting a specific simphony version for testing to avoid the false positives that are due to recent changes in simphony common. Thus the PR can be reviewed tested and merged without problems.

At the same time by having the cutting edge built at the same time you can check to see what breaks with the new version of simphony and thus make a separate PR making the necessary changes.

As a result you separate the concepts of writing code to add a feature ro fix a bug in simphony-opefoam and writing code to support a new version of simhony.

@itziakos
Copy link
Member Author

@khiltunen is this ok to merge?

@khiltunen
Copy link
Contributor

Yes.

@itziakos
Copy link
Member Author

ok, I will update the branch to let travis-ci run one more time and then merge.

Thanks

@itziakos
Copy link
Member Author

@khiltunen, I think that his PR might conflict with #23, so I think that it is better if we revisit this after #23 is merged

@khiltunen
Copy link
Contributor

Your are right. Let's wait for the #23.

@stefanoborini
Copy link
Contributor

Closing as it's no longer relevant.

@stefanoborini stefanoborini deleted the matrix-build-travis-ci branch March 10, 2017 16:56
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.

Use specific versions for package depedencies
3 participants