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
Psalm 5 #293
Conversation
Btw you can cherry-pick 7a1c3fd to pull in my optimization fixes if you want to finish this PR, I won't have much time today to work on the PR :) |
Have some severe doubts about how you removed those taint tests, the problem here is that Psalm is not detecting the issue (false negative), not that it shouldn't detect it (false positive). |
thank you very much! It seems the commit is reverting Immutable/Mutable Union types which IMO is a downgrade. |
Taint tests are not removed except one (not removed but commented out temporarily) which does not work correctly due to Psalm bug |
Yeah, there's a psalm bug in taint detection that should be fixed before merging this MR imo |
fixes #285 |
No description provided.