Failure

User tests: Successful: Unsuccessful:

avatar b2z
b2z
12 Feb 2017

Pull Request for Issue #935.

Summary of Changes

vagrant@contrib-jessie:~$ php -v
PHP 7.0.15-1~dotdeb+8.1 (cli) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2017 Zend Technologies
with Zend OPcache v7.0.15-1~dotdeb+8.1, Copyright (c) 1999-2017, by Zend Technologies
with Xdebug v2.5.0, Copyright (c) 2002-2016, by Derick Rethans

Testing Instructions

You will need to destroy your current virtual machine and create a new one by running vagrant up.

avatar b2z b2z - open - 12 Feb 2017
avatar b2z
b2z - comment - 12 Feb 2017

Should I provide a manual how to update our existing box to PHP 7?

avatar mbabker
mbabker - comment - 12 Feb 2017

Should I provide a manual how to update our existing box to PHP 7?

Is it really necessary? I don't actually use Vagrant much so I'm not sure what all this would entail or what help it would be.

avatar b2z
b2z - comment - 12 Feb 2017

Dunno, just asking :) BTW what is PHP nightly Travis is not happy of?

avatar mbabker
mbabker - comment - 12 Feb 2017

Wouldn't worry about it too much. Basically tests against PHP's master branch. Seems like something in PHPCS doesn't work so well with 7.2 right now.

avatar mbabker mbabker - change - 12 Feb 2017
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-12 18:52:20
Closed_By mbabker
avatar mbabker mbabker - close - 12 Feb 2017
avatar mbabker mbabker - merge - 12 Feb 2017
avatar mbabker mbabker - reference | 25e6363 - 12 Feb 17
avatar mbabker mbabker - merge - 12 Feb 2017
avatar mbabker mbabker - close - 12 Feb 2017
avatar mbabker mbabker - head_ref_deleted - 12 Feb 2017
avatar b2z
b2z - comment - 12 Feb 2017

Ah ok :)

avatar b2z
b2z - comment - 12 Feb 2017

So we are rising our requirement to PHP 7.0 or later? ;)

avatar mbabker
mbabker - comment - 12 Feb 2017

Yep. The server's been running it for some time, time for us to catch up.

Add a Comment

Login with GitHub to post a comment