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

Upgrade to Laravel 10 #10247

Merged
merged 3 commits into from
Jul 6, 2023
Merged

Upgrade to Laravel 10 #10247

merged 3 commits into from
Jul 6, 2023

Conversation

nanaya
Copy link
Collaborator

@nanaya nanaya commented May 25, 2023

  • reset middleware order (to match existing behaviour; should probably readd it later)
  • reset connections when dealing with database outside normal test cycle already done in a separate PR

Seems working but probably can use a bit more tests 🤔

Resolves #10042.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
westonruter Weston Ruter
- reset middleware order (to match existing behaviour; should probably readd it later)
Copy link
Collaborator

@notbakaneko notbakaneko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

working so far 👀

@notbakaneko notbakaneko enabled auto-merge July 6, 2023 08:01
@notbakaneko notbakaneko disabled auto-merge July 6, 2023 08:02
@nanaya nanaya merged commit 74d57c2 into ppy:master Jul 6, 2023
@nanaya nanaya deleted the laravel10 branch August 1, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade Laravel
2 participants