Skip to content
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

Enhancement: Use phpstan/phpstan instead of vimeo/psalm #167

Merged
merged 1 commit into from
Jan 5, 2025

Conversation

localheinz
Copy link
Contributor

This pull request

  • uses phpstan/phpstan instead of vimeo/psalm

@localheinz localheinz self-assigned this Jan 5, 2025
@localheinz localheinz force-pushed the feature/phpstan branch 2 times, most recently from 020fe85 to a1f08cb Compare January 5, 2025 14:10
@localheinz localheinz merged commit 26be97b into main Jan 5, 2025
2 checks passed
@localheinz localheinz deleted the feature/phpstan branch January 5, 2025 14:11
@drupol
Copy link

drupol commented Jan 5, 2025

Hello Andrea,

Happy new year mate :-)

It's not the first time I see this kind of PR and I'm wondering why this mass exodus of users to PHPStan in place of Psalm. Can you enlighten me?

Thank you!

@localheinz
Copy link
Contributor Author

A happy new year to you, too, @drupol!

There has not been much activity for vimeo/psalm lately, so I suspect - although I have been a fan of it - that it's dead.

More importantly, vimeo/psalm does not support PHP 8.4 yet, see

@drupol
Copy link

drupol commented Jan 7, 2025

Oooow OK :( Thank you for letting me know... I was not aware since I haven't been in the PHP world since a couple of months now.

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

Successfully merging this pull request may close these issues.

2 participants