-
Notifications
You must be signed in to change notification settings - Fork 4
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
Finds new something
in nowdocs/heredocs
#1
Comments
That reminds me again that I always wanted to migrate the tool to use php_parser instead of doing everything by hand with regexes. Shouldn't be too hard and also would catch all of these cases that don't happen in our codebase like this one. Unfortunately I don't know when I'll have time for it, especially since it's no pressing issue for us at the moment as the tool does what it is supposed to do very well on our codebase ;) |
Is this still valid? It looks like you're using PHPParser now. |
Ah... I just noticed you're including the PHPParser in composer but it doesn't look like any of the code is actually using it. :/ |
Unfortunately I did not get any further then this so far |
When i run the tool on files with nowdocs/heredocs containing the words
new foo
then it thingsfoo
is a classname that is undefined.The text was updated successfully, but these errors were encountered: