search

php / pie

The PHP Installer for Extensions
BSD 3-Clause "New" or "Revised" License
630 stars 18 forks source link

Use `#[RequiresOperatingSystemFamily]` where possible #112

Closed alexandre-daubois closed 3 days ago

alexandre-daubois commented 3 days ago

It is easier to find all tests where Windows is required, as self::markTestSkipped() messages are not 100% consistent across all calls. Also it should be better for PHPUnit so it doesn't have to start executing the test to skip it.

asgrim commented 3 days ago

Thanks @alexandre-daubois