Update percymamedy/laravel-dev-booter to the latest version still compatible with Laravel 10.x
Remove the unused dependency percymamedy/laravel-dev-booter
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T360023 ⏫ Update to Laravel 11 | |||
| Resolved | Ollie.Shotton_WMDE | T424469 ⏫ Remove percymamedy/laravel-dev-booter |
Event Timeline
Comment Actions
Currently, v4.0.0 of percymamedy/laravel-dev-booter is installed. The next version is v5.0.0, however that isn't possible to update to while we are still on Laravel v10.x.
$ docker run --rm -it -v $PWD:/app -u $(id -u):$(id -g) composer:2.9.2 --ignore-platform-req= ext-pcntl --with-all-dependencies require percymamedy/laravel-dev-booter:^5.0 ./composer.json has been updated Running composer update percymamedy/laravel-dev-booter --with-all-dependencies Loading composer repositories with package information Updating dependencies Your requirements could not be resolved to an installable set of packages. Problem 1 - Root composer.json requires percymamedy/laravel-dev-booter ^5.0 -> satisfiable by percymamedy/laravel-dev-booter[v5.0.0]. - percymamedy/laravel-dev-booter v5.0.0 requires illuminate/support ^11.0 -> found illuminate/support[v11.0.0, ..., v11.51.0] but these were not loaded, likely because it conflicts with another require. Installation failed, reverting ./composer.json and ./composer.lock to their original content.
NOTE: percymamedy/laravel-dev-booter hasn't been updated since 2024-07-11, and doesn't currently support Laravel 12.x. We may want to reconsider using it.
Comment Actions
Did some more investigation, we register the ServiceProvider in config/app.php, and have some default config in config/dev-booter.php, but never actually use it. See this code search of wbstack/api.
It stopped being used in fix(local): adjust use of local dev dependenices (#648).
Moving back to Todo to rip it out.
Comment Actions
outdooracorn opened https://github.com/wbstack/api/pull/1113
Remove unused percymamedy/laravel-dev-booter dependency
Comment Actions
Stalled simply waiting for T424471 to be merged to avoid rebasing the very complex passport patch
Comment Actions
Rebased on main now that T424471: ⏫ Update laravel/passport has been merged. Ready for review.
Comment Actions
outdooracorn closed https://github.com/wbstack/api/pull/1113
Remove unused percymamedy/laravel-dev-booter dependency