-
Notifications
You must be signed in to change notification settings - Fork 205
PHP 8.5 support #368
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
PHP 8.5 support #368
Conversation
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
|
Just verifying: all of these changes are related to PHP 8.5? 38 files changes seems like a lot, were the directories and file renaming part of 8.5 specifically? |
@joetannenbaum Yes, for PHP 8.5, we have a dependency on PEST 4 which is causing many changes. If you’d like, I can break this PR down into individual package updates and then ship only php8.5 related changes. |
Key Changes
PHP 8.5 Support: Added to CI test matrix and created AI guidelines for new features of PHP 8.5
Dependency Updates:
laravel/mcp→^0.4.0pestphp/pest→ Added^4.1.5supportlaravel/pint→^1.20.0Test Fixtures Refactor: Renamed
tests/fixtures/→tests/Fixtures/fixture()function that expects aFixtures/directory by defaultfixture()for paths andfixtureContent()for file contents to align with Pest 4 conventions with support for pest 3 and pest 2MCP Tools: Updated all tool classes to use new JsonSchema contract API (
Illuminate\Contracts\JsonSchema\JsonSchemaandTypeclass)