Page MenuHomePhabricator

[HHVM] Tests on Travis are failing (due to PHPUnit version used)
Closed, ResolvedPublic

Description

This is due to the PHPUnit version used.


Version: unspecified
Severity: normal
Whiteboard: u=dev c=backend p=5

Details

Reference
bz69524

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:28 AM
bzimport set Reference to bz69524.
bzimport added a subscriber: Unknown Object (MLST).

We are using the TravisCI default, which is PHPUnit 4.x. I recall Sebastian saying he fixed HHVM compat, so I'm not sure what the issue there is. Will investigate.

Core uses PHPUnit too, and its unit tests are passing with Travis CI under HHVM.

Unless there are breaking changes we made for HHVM support, I suggest be backport the changes onto the 1.x branch and do a 1.1.0 release rather than already releasing 2.0. I do not think a breaking release otherwise justifies the hassle it causes.

Lydia_Pintscher removed a subscriber: Unknown Object (MLST).
Lydia_Pintscher removed a subscriber: Unknown Object (MLST).Dec 1 2014, 2:32 PM