Conversation
There was a problem hiding this comment.
Pull request overview
This PR modernizes the project’s test/tooling setup by migrating the existing PHPUnit-style test class to Pest, introducing baseline Rector configuration, and updating PHPUnit configuration to align with newer PHPUnit versions.
Changes:
- Add Pest bootstrap (
tests/Pest.php) and convertBinaryFlagsTestfrom aPHPUnit\Framework\TestCaseclass to Pest tests. - Add a
rector.phpconfiguration for applying Rector tosrc/andtests/. - Update
phpunit.xmlfor PHPUnit 10-style configuration and adjustcomposer.jsondev tooling dependencies (Pest/Pint/Rector) and PHP version constraints.
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/Pest.php | Adds Pest bootstrap configuration for the test suite. |
| tests/BinaryFlagsTest.php | Migrates the test suite from PHPUnit class methods to Pest DSL tests. |
| rector.php | Introduces Rector config targeting src/ and tests/. |
| phpunit.xml | Updates PHPUnit configuration to the PHPUnit 10 schema and <source> layout. |
| composer.json | Updates dev dependencies and changes minimum PHP/platform configuration. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 35 out of 36 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 35 out of 36 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary