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

Feature request: Check dump() im Twig template files #15

Open
umutphp opened this issue Jun 16, 2021 · 2 comments
Open

Feature request: Check dump() im Twig template files #15

umutphp opened this issue Jun 16, 2021 · 2 comments

Comments

@umutphp
Copy link
Contributor

umutphp commented Jun 16, 2021

An issue was already created on the former repo by @HarasimowiczKamil . Aditional to this issue, a new issue is also created in the umutphp/php-var-dump-check-action by @ajakov. So, I think It is worth discussing to add support to check the non-php file formats.

@ajakov
Copy link

ajakov commented Jun 16, 2021

Thank you @umutphp for raising the issue. I am also willing to contribute.
Do you have some guidelines on what would be the easiest / proper way to do it?

@umutphp
Copy link
Contributor Author

umutphp commented Jun 18, 2021

@ajakov

token_get_all does not support Twig files, so, imho the proper way seems to be a regular expression check in files. But, I am not sure if the maintainers prefers that way.

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

No branches or pull requests

2 participants