Skip to content
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

Include vendor deprecation warnings #124

Open
arderyp opened this issue Feb 23, 2017 · 4 comments
Open

Include vendor deprecation warnings #124

arderyp opened this issue Feb 23, 2017 · 4 comments

Comments

@arderyp
Copy link

arderyp commented Feb 23, 2017

Firstly, thanks for the awesome tool!

I've run it against my codebase with:

./deprecation-detector check $PROJECT/src $PROJECT/vendor

It found lots of helpful things to cleanup. Now when I run it, nothing is found. However, if I go to one of my views, I do see deprecation warnings in the profiler related to my Twig dependency. It's not my issue, but the related deprecation is discussed here: twigphp/Twig#2267

If there is any more info I can provide that would be helpful, please let me know.

@slde-robin
Copy link
Contributor

Actually the tool doesn't check if there are any deprecations in your twig templates.

@arderyp
Copy link
Author

arderyp commented Feb 28, 2017 via email

@slde-robin
Copy link
Contributor

slde-robin commented Mar 1, 2017

The profiler gathers the warnings by actually executing the code.
The detector does not execute the code.

To add Twig support we would need a twig parser and so on... IMO thats too much.

But you could check your compiled Twig templates in the cache dir for deprecations. Right now I don't have the time to test that myself. It would be great if you could test that and give me review

@arderyp
Copy link
Author

arderyp commented Mar 1, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants