gave feedback on sebastianbergmann/php-code-coverage#694 which should be fixed with the UMD version of Popper
In PhpStorm, the .yml schema validation complains about this line: https://github.com/sebastianbergmann/phpunit/blob/master/.travis.yml#L55
env: php-cs-fixer
It expects the environment variable having a value. Would it make sense to give it a value (e.g., 1) to get rid of this validation warning?
env: php-cs-fixer
It expects the environment variable having a value. Would it make sense to give it a value (e.g., 1) to get rid of this validation warning?
ahja! ich vermute: RegEx Elend
the whole @annotation parsing could use a nice cleanup
while digging around
@dataProvider I found some lookups being done twice, for example
@epdenouden do we haz issue for annotations? I can look into it.
it's one of those older parts that 'just kinda works'
work with a rather high "initial : bugfix" pull request ratio
@Ocramius Most of the annotation processing happens in https://github.com/sebastianbergmann/phpunit/blob/master/src/Util/Test.php
I'm fiddling around with
@depends anyway at the moment
Poking again: need review on sebastianbergmann/phpunit#3810 - it has one discussion point, but otherwise done-ish
@bofalke seems like a bad idea: can expect one of PHPUnit's converted exception types
->expectException(\PHPUnit\yaddayadda\WarningThingy) works just fine
@SebastianKull see https://www.php.net/manual/en/language.types.float.php so it makes sense in the validation of equality (precision depends on the system, use bcmath, gmp)
