Success

User tests: Successful: Unsuccessful:

avatar yvesh
yvesh
15 Aug 2016

Summary of Changes

Fixes Travis tests in weblinks.

Testing Instructions

Review travis log

@puneet0191 @rdeutz please merge ASAP so travis starts working again.

avatar yvesh yvesh - open - 15 Aug 2016
avatar puneet0191
puneet0191 - comment - 15 Aug 2016

Thanks @yvesh Looks good. Passes on Travis. Merging.

avatar rdeutz
rdeutz - comment - 15 Aug 2016

@puneet0191 you're fast.

Tried it and it still fails: https://travis-ci.org/rdeutz/weblinks/builds/152374721

avatar puneet0191
puneet0191 - comment - 15 Aug 2016

@rdeutz it is a false positive, not related to the PR. Can you try to restart the job of PHP 5.5? It should pass.

avatar rdeutz
rdeutz - comment - 15 Aug 2016

restarting is what I am always doing when just one PHP Version is failing, I did it again and ups it is still failing

avatar puneet0191
puneet0191 - comment - 15 Aug 2016

mmh...Something to be fixed in the test, Let me create a new issue so that we review the test and avoid the failure for System Message. Thanks

avatar yvesh
yvesh - comment - 15 Aug 2016

@rdeutz Hmm the one from the PR and these from my repo are green:

https://travis-ci.org/joomla-extensions/weblinks/builds/152369889

The system tests in weblinks sometimes have hiccups. We are working on improving that :-)

avatar rdeutz
rdeutz - comment - 15 Aug 2016

You said test and I am just reporting my findings :-)

avatar yvesh
yvesh - comment - 15 Aug 2016

@rdeutz Hehe, i didn't want to say that we will ignore your findings :) Puneet opened an issue and i am going to look into it in the next days.

Add a Comment

Login with GitHub to post a comment