Magentron/laravel-blade-lint

Use stdin instead of writing a temp file

Closed this issue · 9 comments

using proc_open we can pipe the content of the compiled string into php -l thus saving us the hassle of creating temporary files.

Leave it to me I'll make a PR 👍

I'm not so sure whether this is actually an improvement as some installations may not have this function available (e.g. shared hosting environments where this has been disabled via disable-functions.

Such environment won't have exec either when you think about it.

And why would you want to run this script on shared hosting in the first place 🤷‍♂️

Makes sense, please submit the PR and I'll review it. Thanks in advance.

On it.

Here it is #3

Thanks! It's now publicly available as 1.0.5

Great thanks 👍