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.

Sign up for GitHub

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

Jump to bottom

chore(deps): use laravel 10 #2036

Closed
wants to merge 3 commits into from
Closed

chore(deps): use laravel 10 #2036

wants to merge 3 commits into from

Conversation

wescopeland
Copy link
Member

This PR migrates RAWeb from Laravel 9 to Laravel 10. This was a mostly seamless transition with only one breaking change.

Summary of changes:

  • Removed all explicit calls to registerPolicies().
  • Set cache:prune-stale-tags to run hourly.
  • Added a few missing imports to some PHP files.
  • Renamed $routeMiddleware to $middlewareAliases.
  • Updated validateEmail.php to account for an API change with the Validator class.

I also set the pre-push hook to use composer paratest instead of composer test.

@wescopeland wescopeland requested review from luchaos and a team November 18, 2023 14:57
@luchaos
Copy link
Member

luchaos commented Nov 18, 2023 •
edited
Loading

i have an open branch for the upgrade which includes a few more things. i can state the PR in a few hours.
paratest has yet to be tested on windows.

@wescopeland
Copy link
Member Author

Sounds good, no worries at all. Will close this one.

@wescopeland wescopeland deleted the laravel-10 branch November 18, 2023 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luchaos luchaos Awaiting requested review from luchaos

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wescopeland @luchaos