Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the PHP version requirement documentation from 8.2+ to 8.4+ in the README to reflect the project's current configuration.
- Updates README.md to document PHP 8.4+ as the minimum requirement
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Getting Started | ||
|
|
||
| This project uses [Laravel 11](https://laravel.com/) (PHP 8.2+) as the backend framework, with [Inertia.js](https://inertiajs.com/) and [Vue 3](https://vuejs.org/) for the frontend. | ||
| This project uses [Laravel 11](https://laravel.com/) (PHP 8.4+) as the backend framework, with [Inertia.js](https://inertiajs.com/) and [Vue 3](https://vuejs.org/) for the frontend. |
There was a problem hiding this comment.
This documentation states that PHP 8.4+ is required, but Laravel 11's official requirement is PHP 8.2+. While the project's composer.json does specify "^8.4", this creates a discrepancy with Laravel's official documentation. If the project intentionally requires PHP 8.4 as a minimum, consider clarifying this is a project-specific requirement rather than a Laravel 11 requirement. Otherwise, consider documenting it as "PHP 8.2+" to align with Laravel 11's actual requirements, while noting the project is configured to use PHP 8.4.
| This project uses [Laravel 11](https://laravel.com/) (PHP 8.4+) as the backend framework, with [Inertia.js](https://inertiajs.com/) and [Vue 3](https://vuejs.org/) for the frontend. | |
| This project uses [Laravel 11](https://laravel.com/) as the backend framework and is configured to run on PHP 8.4+ (Laravel 11 itself requires PHP 8.2+ or higher), with [Inertia.js](https://inertiajs.com/) and [Vue 3](https://vuejs.org/) for the frontend. |
1 participant
No description provided.