Skip to content

Feature/17#18

Open
rochamarcelo wants to merge 4 commits intodakota:masterfrom
promo-sapiens:feature/17
Open

Feature/17#18
rochamarcelo wants to merge 4 commits intodakota:masterfrom
promo-sapiens:feature/17

Conversation

@rochamarcelo
Copy link
Copy Markdown

For issue #17 . Please let me know if this is okay.

Thanks.

@rochamarcelo
Copy link
Copy Markdown
Author

The recent version of CakePHP requires php >= 5.6

@rochamarcelo
Copy link
Copy Markdown
Author

@josegonzalez is this an issue with phpunit? What should I do to fix it?

@josegonzalez
Copy link
Copy Markdown
Collaborator

@ADmad do you know how to fix this sort of issue?

@elboletaire
Copy link
Copy Markdown

elboletaire commented Jul 12, 2018

Have you find out what's happening with travis so this can be finally merged?

I've been checking it a bit and, in phpunit_aliases.php there's a requirement of phpunit >= 5.7:

if (version_compare(\PHPUnit_Runner_Version::id(), '5.7', '<')) {
    trigger_error(sprintf('Your PHPUnit Version must be at least 5.7.0 to use CakePHP Testsuite, found %s', \PHPUnit_Runner_Version::id()), E_USER_ERROR);
}

and travis is running phpunit 5.6... But the error is not triggered.. and, instead it's trying to load the class aliases.

Weird 😕

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.

3 participants