Skip to content

Do not throw exceptions on deprecation errors while autoloading deps#11313

Merged
danog merged 3 commits intovimeo:6.xfrom
danog:improve_error_handling
Feb 18, 2025
Merged

Do not throw exceptions on deprecation errors while autoloading deps#11313
danog merged 3 commits intovimeo:6.xfrom
danog:improve_error_handling

Conversation

@danog
Copy link
Collaborator

@danog danog commented Feb 18, 2025

No description provided.

@danog danog added the release:fix The PR will be included in 'Fixes' section of the release notes label Feb 18, 2025
@danog danog changed the title Do not throw exceptions on deprecation errors in project dependencies Do not throw exceptions on deprecation errors while autoloading deps Feb 18, 2025
@danog danog merged commit c70baa4 into vimeo:6.x Feb 18, 2025
50 of 51 checks passed
@Yi-pixel
Copy link

This change appears to cause the following error when executing

$ ./vendor/bin/psalm-plugin enable psalm/plugin-laravel

PHP Fatal error:  Uncaught Error: Class "Psalm\Internal\ErrorHandler" not found in /var/www/html/vendor/vimeo/psalm/src/Psalm/Internal/CliUtils.php:136
Stack trace:
#0 /var/www/html/vendor/vimeo/psalm/src/Psalm/Internal/Cli/Plugin.php(32): Psalm\Internal\CliUtils::requireAutoloaders()
#1 /var/www/html/vendor/vimeo/psalm/psalm-plugin(9): Psalm\Internal\Cli\Plugin::run()
#2 /var/www/html/vendor/bin/psalm-plugin(119): include('...')
#3 {main}
  thrown in /var/www/html/vendor/vimeo/psalm/src/Psalm/Internal/CliUtils.php on line 136

$ composer show|grep psalm
psalm/plugin-laravel                      2.12.1
psalm/plugin-mockery                      1.2.0
psalm/plugin-phpunit                      0.19.2
vimeo/psalm                               6.8.4

$ php -v
php 8.4.3

and, psalm/plugin-laravel No longer working

@danog
Copy link
Collaborator Author

danog commented Feb 21, 2025

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:fix The PR will be included in 'Fixes' section of the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants