-
Notifications
You must be signed in to change notification settings - Fork 678
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
Support references #7254
Labels
Comments
I found these snippets: https://psalm.dev/r/d79b9b2f0f<?php
$a = 1;
$b = &$a;
$b = 2;
$c = 3;
$b = &$c;
$b = 4;
/** @psalm-trace $a, $b, $c */;
|
Yeah, I believe we also have some flaws about references wrongly considered unused. I tried to flag issues related to references here: https://github.com/vimeo/psalm/issues?q=is%3Aissue+is%3Aopen+label%3Areferences |
AndrolGenhald
added a commit
to AndrolGenhald/psalm
that referenced
this issue
Jan 7, 2022
AndrolGenhald
added a commit
to AndrolGenhald/psalm
that referenced
this issue
Jan 7, 2022
AndrolGenhald
added a commit
to AndrolGenhald/psalm
that referenced
this issue
Jan 11, 2022
AndrolGenhald
added a commit
to AndrolGenhald/psalm
that referenced
this issue
Jan 26, 2022
orklah
added a commit
that referenced
this issue
Jan 26, 2022
Add support for references and improve UnusedVariable checks (fixes #7254).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
References aren't really supported by Psalm at the moment outside of
ReferenceConstraintViolation
and related issues when using references in arguments. It would be nice to fully support references.https://3v4l.org/mQ809
https://psalm.dev/r/d79b9b2f0f
See also #7252 (when fixed skipped tests should be unskipped).
The text was updated successfully, but these errors were encountered: