Add satooshi/php-coveralls to require dev

[codeliner]

Let's see if it fixes #2

[codeliner]

@sandrokeil Can you take a look? Error #2 still exists. Not sure, how to solve it.

[sandrokeil]

@codeliner This is not the solution. We have to remove the src_dir defintion. Seems it's not necessary anymore.

[codeliner]

ok, but the same config works for our other components. Do we need to add the dependency? It is missing in current composer.json

[prolic]

src_dir is not necessary, right. See travis on php 5.6:

[Symfony\Component\Config\Definition\Exception\InvalidConfigurationException]  

Unrecognized option "src_dir" under "coveralls"

Coveralls has changed this with 1.0 release (only a few days old).

[prolic]

My question, why is coveralls only executed on 5.6?

[sandrokeil]

@prolic We need only one analysis, because there is no difference between the PHP versions. coveralls is loaded in the travis.yml. However, we should pin it to version 1.0 there.

[prolic]

Another thing:

.travis.yml should be like this:

after_success:

• travis_retry php vendor/bin/coveralls -v

we use after_script, which is always executed.

[codeliner]

everything set, let's see if it works

[codeliner]

new error: [Symfony\Component\Config\Definition\Exception\InvalidConfigurationExcept
ion]
coverage_clover XML file is not readable

[codeliner]

I'm going to merge this PR. @sandrokeil can you try to fix it with a new PR then? I need to finish today.

[sandrokeil]

@codeliner I will check it

[prolic]

build/logs/clover.xml is wrong in https://github.com/prooph/psr7-middleware/blob/master/composer.json#L70

[codeliner]

thx

[sandrokeil]

@prolic My fault, i will fix it. Or you will do. ;-)