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

Add CI helper script invocation to Travis CI #627

Closed

Conversation

martbab
Copy link
Contributor

@martbab martbab commented Mar 20, 2017

This tests whether changes in ipatests do not break any of the helper scripts
used for integration testing. The PR is rebased on
#626 so it should produce green build
with @cheimes's fixes

@martbab martbab force-pushed the travis-ci-integration-helpers branch from 2698f71 to 469db50 Compare March 20, 2017 14:52
@tkrizek
Copy link
Contributor

tkrizek commented Mar 22, 2017

Could you please re-rebase this on top of #626 to check if everything works as expected?

@martbab martbab force-pushed the travis-ci-integration-helpers branch from 469db50 to 964a795 Compare March 22, 2017 11:51
@martbab
Copy link
Contributor Author

martbab commented Mar 22, 2017

@tomaskrizek I guess Green Travis means we can merge #626 and I can then remove those commits from this PR right?

@tkrizek
Copy link
Contributor

tkrizek commented Mar 22, 2017

@martbab Exactly, please remove the extra commits.

This is to verify that all the executables are working properly (have
correct import paths etc.)
@martbab martbab force-pushed the travis-ci-integration-helpers branch from 964a795 to aec9f5e Compare March 22, 2017 12:55
@tkrizek tkrizek added the ack Pull Request approved, can be merged label Mar 22, 2017
@tkrizek
Copy link
Contributor

tkrizek commented Mar 22, 2017

master:

  • b662459 Travis CI: invoke integration test helper scripts before test execution

@tkrizek tkrizek added the pushed Pull Request has already been pushed label Mar 22, 2017
@tkrizek tkrizek closed this Mar 22, 2017
@martbab martbab deleted the travis-ci-integration-helpers branch March 22, 2017 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants