dorgflow icon indicating copy to clipboard operation
dorgflow copied to clipboard

Show php 7 as requirement

Open theodorosploumis opened this issue 8 years ago • 3 comments
trafficstars

Added php 7 as requirement and show better info for installation.

theodorosploumis avatar Mar 02 '17 15:03 theodorosploumis

The script itself doesn't need PHP 7, it's just Composer which is being rubbish -- see https://github.com/joachim-n/dorgflow/issues/11

joachim-n avatar Mar 02 '17 16:03 joachim-n

Actually it needs php 7. For example this line is using the new Spaceship operator. I managed to install the library with phpunit 5.7 but then I got to this line.

theodorosploumis avatar Mar 02 '17 16:03 theodorosploumis

Ah, I'd completely forgotten about the use of the spaceship!

Wouldn't the PHP version requirement be better done in the composer.json file?

Also, I don't think there's a need to give git cloning instructions -- github has those.

joachim-n avatar Mar 04 '17 23:03 joachim-n