Skip to content

Draft: Re-enable installation of latest phpunit/php-code-coverage:^9.2.21#1777

Merged
maks-rafalko merged 1 commit intoinfection:masterfrom
Ocramius:fix/#1773-re-enable-latest-php-code-coverage-installation
Dec 14, 2022
Merged

Draft: Re-enable installation of latest phpunit/php-code-coverage:^9.2.21#1777
maks-rafalko merged 1 commit intoinfection:masterfrom
Ocramius:fix/#1773-re-enable-latest-php-code-coverage-installation

Conversation

@Ocramius
Copy link
Copy Markdown
Contributor

@Ocramius Ocramius commented Dec 14, 2022

This change reverts some of #1773 by allowing the latest PHPUnit coverage tooling to be installed, for which @Slamdunk fixed coverage in regards of the infection/infection use-case.

Ref: https://github.com/sebastianbergmann/php-code-coverage/releases/tag/9.2.21
Ref: sebastianbergmann/php-code-coverage#964
Ref: sebastianbergmann/php-code-coverage@71525ea

@Ocramius
Copy link
Copy Markdown
Contributor Author

Uh-oh, composer install failed with Error: Process completed with exit code 123. @_@

@Ocramius Ocramius changed the title Re-enable installation of latest phpunit/php-code-coverage:^9.2.20 Draft: Re-enable installation of latest phpunit/php-code-coverage:^9.2.20 Dec 14, 2022
This change reverts some of infection#1773 by allowing the latest PHPUnit coverage tooling to
be installed, for which @Slamdunk fixed coverage in regards of the `infection/infection`
use-case.

Ref: https://github.com/sebastianbergmann/php-code-coverage/releases/tag/9.2.21
Ref: sebastianbergmann/php-code-coverage#964
Ref: sebastianbergmann/php-code-coverage@71525ea
@Ocramius Ocramius force-pushed the fix/#1773-re-enable-latest-php-code-coverage-installation branch from 2d03b1d to 69a1b01 Compare December 14, 2022 13:31
@Ocramius Ocramius changed the title Draft: Re-enable installation of latest phpunit/php-code-coverage:^9.2.20 Draft: Re-enable installation of latest phpunit/php-code-coverage:^9.2.21 Dec 14, 2022
@maks-rafalko maks-rafalko enabled auto-merge (squash) December 14, 2022 15:41
@maks-rafalko maks-rafalko merged commit b6ae298 into infection:master Dec 14, 2022
@maks-rafalko
Copy link
Copy Markdown
Member

Thank you @Ocramius

@sanmai
Copy link
Copy Markdown
Member

sanmai commented Dec 16, 2022

Looks like we are not in the clear yet: #1778

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.

4 participants