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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[9.x] Test for @php blade tag 馃攳 #45501

Merged
merged 1 commit into from Jan 4, 2023

Conversation

imanghafoori1
Copy link
Contributor

This missing edge case test for @php blade tags was the reason behind #45388 and introducing the breaking change in PR: #45333

So having it avoids reintroducing the breaking change again. I added this to complement the #45490 since that one also changes the blade compiler.

@taylorotwell taylorotwell merged commit 140eb47 into laravel:9.x Jan 4, 2023
@imanghafoori1 imanghafoori1 changed the title [9.x] Add test for @php blade tag [9.x] Test for @php blade tag 馃攳 Jan 10, 2023
@imanghafoori1 imanghafoori1 deleted the php_blade branch January 23, 2023 07:36
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

Successfully merging this pull request may close these issues.

None yet

2 participants