You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If we instead throw a PHPStan\AnalysedCodeException we could report issues like these back to the user.
I gave it a quick go and it looked to work well, the only issue was that I couldn't figure out how to make this test happy since it tries to include a non existing file and I can't figure out what path it is looking in so couldn't add a file for it either.
Currently the package simply ignores when a template cannot be found and just assumes it's an empty file.
bladestan/src/Compiler/BladeToPHPCompiler.php
Line 111 in ff27832
If we instead throw a
PHPStan\AnalysedCodeException
we could report issues like these back to the user.I gave it a quick go and it looked to work well, the only issue was that I couldn't figure out how to make this test happy since it tries to include a non existing file and I can't figure out what path it is looking in so couldn't add a file for it either.
https://github.com/TomasVotruba/bladestan/blob/main/tests/Compiler/BladeToPHPCompiler/Fixture/basic_include.blade.php#L1
But maybe it would also be better to not stop processing but simply report the issue and continue as we du currently. But I don't know how to do so 😅
The text was updated successfully, but these errors were encountered: