-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Remove PHPCov dependency via VCS composer repository #6734
Comments
We don't rely on it, but back then, there was no packagist entry. |
I suppose we can bump to |
AFAIK the command line arguments changed in the 2.* series, so we also need to update the calls to it then. |
as I tried at #6835 by issue I created at #6829 for coveralls error, when using
as @DASPRiD said,
and re-tried by using
and then the coveralls won't generated. |
We should probably kill coveralls and use https://github.com/scrutinizer-ci/ocular instead |
@Ocramius do zf2 repo already registered at scrutinizer ? |
Not by the organization :-\ |
anyone have access to register it ? |
@samsonasik only @weierophinney can do that, though I can probably bribe @schmittjoh <_< |
:) We could add it ofc, but it would never fully work without someone setting it up with admin access to the Zend repository. To just get results once, I'd rather suggest running Scrutinizer on a fork. |
@schmittjoh I'll blackmail people in the Zend team then :-) |
Indeed, thanks! |
See https://github.com/zendframework/zf2/blob/f77f90bfe635a762d1a89469fe846585d636a89f/composer.json#L11
Unless we REALLY need the exact version of
sebastianbergmann/phpcov:1.1.0
, then this dep can go away, and we can rely on the packagist repophpunit/phpcov
Assigning to @DASPRiD since I can't work on this until monday.
/cc @Seldaek
The text was updated successfully, but these errors were encountered: