Skip to content

PHPstan integration #554

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

Open
rikvdh opened this issue Feb 24, 2025 · 1 comment
Open

PHPstan integration #554

rikvdh opened this issue Feb 24, 2025 · 1 comment

Comments

@rikvdh
Copy link

rikvdh commented Feb 24, 2025

Is your feature request related to a problem? Please describe.
No

Describe the solution you'd like
Since I need to move away from ext-imap and also from laminas/mail I'm evaluating options for IMAP interaction in PHP.

For our project we want to serve a high quality level. I was testing this library against phpstan (static analyzer). I found a lot of potential problems. I was working on some of them already. But before continuing my work I want to discuss this as this will come all together as a big change.

See my current change-set:
https://github.com/Webklex/php-imap/compare/master...rikvdh:php-imap:feature/phpstan-integration?expand=1

(I'm not done yet)

Describe alternatives you've considered
Reason to move away from ext-imap:
php/pecl-mail-imap#15

Reason to move away from laminal/mail:
laminas/laminas-mail#269

@rikvdh
Copy link
Author

rikvdh commented Mar 10, 2025

@Webklex Is phpstan support something you want integrated? Then I can continue my work on this.

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

No branches or pull requests

1 participant