Skip to content

4268: Replaced Psaml with PHPStan #19

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
wants to merge 9 commits into
base: feature/4268-api-platform-v4
Choose a base branch
from

Conversation

rimi-itk
Copy link

@rimi-itk rimi-itk commented Jun 4, 2025

Builds on #18 to replace Psalm with PHPStan (via https://github.com/phpstan/phpstan-symfony).

Due to lazyness we don't (yet) care about specifying types in arrays and iterables and hence ignore related errors in phpstan.dist.neon.

Furthermore, we ignore some errors related to $id properties in DTOs since these are actually set by API Platform.

@rimi-itk rimi-itk force-pushed the feature/4268-api-platform-v4-phpstan branch from ccd986d to a1666ca Compare June 4, 2025 14:59
@rimi-itk rimi-itk force-pushed the feature/4268-api-platform-v4-phpstan branch from 6eb65bc to 263c400 Compare June 4, 2025 21:07
@rimi-itk rimi-itk marked this pull request as ready for review June 4, 2025 22:26
@rimi-itk rimi-itk force-pushed the feature/4268-api-platform-v4-phpstan branch from b2d9c0d to 8957545 Compare June 4, 2025 22:39
@rimi-itk rimi-itk requested a review from turegjorup June 4, 2025 22:41
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

Successfully merging this pull request may close these issues.

2 participants