Skip to content

switched to drupal_ti#5

Open
LKS90 wants to merge 2 commits intomd-systems:8.x-1.xfrom
LKS90:switchToDrupalTI
Open

switched to drupal_ti#5
LKS90 wants to merge 2 commits intomd-systems:8.x-1.xfrom
LKS90:switchToDrupalTI

Conversation

@LKS90
Copy link
Contributor

@LKS90 LKS90 commented Aug 5, 2015

No description provided.

@LKS90
Copy link
Contributor Author

LKS90 commented Aug 5, 2015

Drupal_TI seems to work, it's complaining about failed tests now. Will take a look tomorrow or something.

Berdir
Berdir reviewed Aug 5, 2015
View reviewed changes
.travis.yml Outdated
Copy link
Member

@Berdir Berdir Aug 5, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

has 5.5 twice?

@LKS90
Fix .travis.yml
9ef2aa4 
Php 5.5 was listed twice, changed the second one to 5.6
@LKS90
Copy link
Contributor Author

LKS90 commented Aug 5, 2015

Like postfinance, can't reproduce. But there is an exception:

Uncaught PHP Exception
    GuzzleHttp\Exception\ConnectException: "cURL error 28: Operation timed out
    after 30032 milliseconds with 0 out of -1 bytes received (see
    http://curl.haxx.se/libcurl/c/libcurl-errors.html)"
    at /home/travis/build/md-systems/drupal-8/drupal/core/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php line 168

There is one usage of guzzle in the SaferpayResponseController, but it's unused (grey and underlined)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@LKS90 @Berdir