Skip to content

fix .travis.yml so that integrationTest does not run if test already failed#633

Merged
awasum merged 1 commit intoapache:developfrom
vorburger:travis-skip-integration-test-if-test-fails
Sep 5, 2019
Merged

fix .travis.yml so that integrationTest does not run if test already failed#633
awasum merged 1 commit intoapache:developfrom
vorburger:travis-skip-integration-test-if-test-fails

Conversation

@vorburger
Copy link
Member

No description provided.

@awasum
Copy link
Contributor

awasum commented Sep 4, 2019

Makes sense. preventing unnecessary execution.

This will really be used once a test fails.

Merge this?

@vorburger
Copy link
Member Author

Merge this?

yes, please! 😸 Tx.

@vorburger
Copy link
Member Author

actually wait a second - have you noticed that Travis hasn't run on this PR? That's weird...

@awasum
Copy link
Contributor

awasum commented Sep 4, 2019

Yeah... maybe leave this and instead merge #634 which contains this commit and one addition commit

@vorburger
Copy link
Member Author

have you noticed that Travis hasn't run on this PR? That's weird...

so Travis runs just fine on #635 .. I think this as is originally is wrong - the \ line break probably doesn't work - let me change this to be all on a single line instead.

@vorburger vorburger force-pushed the travis-skip-integration-test-if-test-fails branch from f90836a to 06ed846 Compare September 4, 2019 20:27
@vorburger
Copy link
Member Author

@awasum fixed it - looks good to you now - merge?

@awasum
Copy link
Contributor

awasum commented Sep 5, 2019

Yes.

@awasum awasum merged commit d9e373e into apache:develop Sep 5, 2019
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