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
Integrate Psalm #10
Integrate Psalm #10
Conversation
This is a generated setup with a few changes: - Reduce the error level from 7 to 3, which finds more issues. - Configure `findUnusedCode="false"` to avoid a warning about it. - Disable extensions "apcu" and "redis". This was recommended by Psalm itself in some warnings that were generated during the GitHub pipeline runs. - Enable FFI extension explicitly. Psalm uses the ones from composer.json by default, but FFI is only marked as a suggestion.
Psalm is almost clueless about FFI and its structure. Support for it was only added recently and is far from complete yet.
fd949fc
to
4955a23
Compare
4955a23
to
7d25bb5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I'd like MrBaulm to also review this 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked out you branch and i can reproduce the named issues and the other issues mentioned in you description. This PR should be merged
Changes:
@psalm-suppress
annotations for FFI stuff. Support for FFI in Psalm is mediocre.Notes:
--show-info=true
, there are some more warnings raised by Psalm, but these are not addressed here.@psalm-suppress
annotations, see Improved FFI integration vimeo/psalm#9230 for further info.