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

Support Laravel 12 #25

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Support Laravel 12 #25

wants to merge 13 commits into from

Conversation

squiaios
Copy link
Contributor

No description provided.

@squiaios squiaios changed the title Laravel 12 Support Laravel 12 Feb 28, 2025
@squiaios
Copy link
Contributor Author

@RobertBoes can you help me with the run-tests.yml file please

I can't configure the excludes correctly for:

  • Laravel version 10 (exclude inertia 2.0)
  • Laravel version 11 (exclude php 8.1)
  • Laravel version 12 (exclude inertia 1.3 & php 8.1)

@RobertBoes
Copy link
Owner

@squiaios I'll take a look 👍🏻

@squiaios
Copy link
Contributor Author

We can remove readonly to keep php 8.1 or not ?

@RobertBoes
Copy link
Owner

RobertBoes commented Feb 28, 2025

You're right, might be nicer to keep 8.1 compatibility

Currently I think the only roadblock is glhd/gretel, as it doesn't have Laravel 12 support yet

@squiaios
Copy link
Contributor Author

You're right, might be nicer to keep 8.1 compatibility

Currently I think the only roadblock is glhd/gretel, as it doesn't have Laravel 12 support yet

didn't check this package, only diglactic ... tabuna works for laravel 12

for glhd, there is a PR : glhd/gretel#22

We can wait to get ghld working on L12

@RobertBoes
Copy link
Owner

Figured I could use a fork to at least know if it'll work, but we'll just have to wait I guess 😅

@RobertBoes
Copy link
Owner

@squiaios Looks like I can't delete that last commit, or even clean up the history. Might also be a skill-issue on my end tho

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