Skip to content

Bump dependency compatibility with laravel 10 #10

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

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

Conversation

alslimany
Copy link

@alslimany alslimany commented Mar 5, 2023

Upgrading dependenies:

  • "inertiajs/inertia-laravel": "^0.6.9"
  • "mockery/mockery": "^1.5.1"
  • "orchestra/testbench": "^6.24.1 || ^7.22.1"
  • "phpunit/phpunit": "^9.6.4",

to be compatible with laravel 10

@frknasir
Copy link

@ksassnowski hi...thank you for this amazing package. I was wondering, will you be merging this soon or are we better off forking it and making the necessary changes ourselves. I am having difficulties upgrading my app to use inertial laravel. Thank you.

@ksassnowski
Copy link
Contributor

Can someone verify for me if the changes in this PR allow you to upgrade to Laravel 10?

@ksassnowski
Copy link
Contributor

@alslimany Can you please also update the GitHub action so it also runs on Laravel 10? https://github.com/laravel-arcanist/inertia-response-renderer/blob/main/.github/workflows/run-tests.yml

@frknasir
Copy link

@ksassnowski looks like that was the last time @alslimany was active on github 😆

@alslimany
Copy link
Author

Hi all, i'm here.
Just waiting to get back home.

@ksassnowski
Copy link
Contributor

Quick heads up that I’m going to be on vacation for a week. I’ll get this merged once in back.

Sorry for the delay.

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.

4 participants