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

Add laravel 11 support #271

Merged
merged 5 commits into from
Feb 26, 2024
Merged

Conversation

mokhosh
Copy link
Contributor

@mokhosh mokhosh commented Feb 23, 2024

No description provided.

@mokhosh
Copy link
Contributor Author

mokhosh commented Feb 23, 2024

workflow checks will fail until spatie/laravel-translatable#434 gets merged.

@mokhosh
Copy link
Contributor Author

mokhosh commented Feb 24, 2024

@freekmurze Is it using php 8.0 and 8.1 even after your change, or are these the old checks?

@mokhosh
Copy link
Contributor Author

mokhosh commented Feb 25, 2024

Uneven indentation was the only difference I could see between your commit here spatie/laravel-translatable@c1426f5 which passed the checks, and this one 7b68b01 which didn't seem to pass.

So I fixed the indentations. Hopefully this will pass. Weird though.

@mokhosh
Copy link
Contributor Author

mokhosh commented Feb 25, 2024

eureka! eureka!
it was a cheeky little comma between 8.1 and 8.0 😅

@mokhosh
Copy link
Contributor Author

mokhosh commented Feb 25, 2024

it's been here for two years 7075e0b

@freekmurze freekmurze merged commit a44afe6 into spatie:main Feb 26, 2024
14 checks passed
@freekmurze
Copy link
Member

Thanks!

@mokhosh mokhosh deleted the add-laravel-11-support branch February 26, 2024 09:49
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.

2 participants