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

[Feature] Upgrade Laravel 8 to 10 #441

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

Conversation

rafaelstz
Copy link

@rafaelstz rafaelstz commented May 14, 2023

WHY are these changes introduced?

This PR upgrades Laravel 8 to 10.

WHAT is this pull request doing?

  • Upgrading Composer packages and dependencies.
  • Removing two third-party packages that became native.
  • Adjusting the link in the README file.
  • Removing PHP 8.0 support from CI.

Checklist

Note: once this PR is merged, it becomes a new release for this template.

  • I have added/updated tests for this change
  • I have made changes to the README.md file and other related documentation, if applicable

@rafaelstz rafaelstz changed the title Upgrade Laravel 8 to 10 [Feature] Upgrade Laravel 8 to 10 May 14, 2023
@grahamsutton
Copy link

grahamsutton commented May 31, 2023

@paulomarg Could we please get this reviewed and merged at your earliest convenience?

Laravel 8 is no longer receiving bug and security fixes as of January 24th, 2023. Laravel 10 will assure everyone is using the latest and most secure version of Laravel available. This should be a high priority item.

https://laravel.com/docs/8.x/releases#support-policy

@simone-boa-ideas
Copy link

simone-boa-ideas commented Jul 17, 2023

@paulomarg @mkevinosullivan Adding my voice to the chorus here :)
Having this would be really great!
The lack of a more updated version of Laravel is actually what's pushing us to use different unofficial solutions like laravel-shopify.
Of course, using the official and maintained version from Shopify would be much better, but we can't compromise on the app security (especially since Laravel 8 is not receiving security fixes anymore)

@angryalf
Copy link

when this be merged?

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.

6 participants