-
Notifications
You must be signed in to change notification settings - Fork 15
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
Treat assert() as failing #68
Comments
Do you think we'll be allowed to change those directly from code or should we ask Matt to change this in the host panel? |
Doc says
So, php.ini change is mandatory |
@muglug could you take a look at this when you have time? It would be great to have psalm.dev behaving correctly when failing an assertion |
I think this is fixed now. @muglug can you confirm you enabled assertions on https://psalm.dev? |
vimeo/psalm#6989 reported a case where psalm is failing due to an assert() in php-parser but psalm.dev continues the analysis. I think psalm.dev should behave as if we were in dev environment because this is usually the way Psalm runs
The text was updated successfully, but these errors were encountered: