-
Notifications
You must be signed in to change notification settings - Fork 158
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
Bump to Laravel v10 & PHP 8.2 #1
Conversation
@jessarcher this is ready but it seems existing tests seem to fail. I wonder if the failure has anything to do with laravel/framework#45500 ? |
@driesvints I fixed the issue by making Testbench load the Livewire service provider so that Livewire is registered. I'm not sure if this is the correct way to solve the issue, but it works 🤷♀️ I've also confirmed that this issue existed prior to this PR. |
@jessarcher lgtm! |
Fix problem doing "composer require laravel/pulse" Uncaught TypeError: Laravel\Pulse\Pulse::Laravel\Pulse\{closure}(): Argument #1 ($event) must be of type Illumi nate\Events\Dispatcher, Illuminate\Events\NullDispatcher given, called in /var/www/sapere/sapere/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 1302 and defined in /var/www/sapere/sapere/vendor/laravel/pulse/src/Pulse.php:119 Stack trace:
Fix problem doing "composer require laravel/pulse" Uncaught TypeError: Laravel\Pulse\Pulse::Laravel\Pulse\{closure}(): Argument #1 ($event) must be of type Illumi nate\Events\Dispatcher, Illuminate\Events\NullDispatcher given, called in /var/www/sapere/sapere/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 1302 and defined in /var/www/sapere/sapere/vendor/laravel/pulse/src/Pulse.php:119 Stack trace:
This PR bumps dependencies to PHP 8.2 & Laravel v10. It also adds the default GitHub files and a CI build.
Depends on: