Skip to content

feat!: upgrade to laravel 13 - #1529

Draft
itinerare wants to merge 1 commit into
lk-arpg:unstablefrom
itinerare:push-kqwnqppuzyyy
Draft

itinerare wants to merge 1 commit into
lk-arpg:unstablefrom
itinerare:push-kqwnqppuzyyy

Conversation

@itinerare

Copy link
Copy Markdown
Member
  • explicitly include all modifiers when changing columns in migrations (required as of Laravel 11)
  • remove security advisory overrides
  • remove doctrine/dbal
  • specify a more precise version for debugbar

Note that Laravel 13 requires PHP 8.3 minimum; this is a breaking change.

Note that while I removed the (now unnecessary) security advisory overrides from composer.json, I did not remove the santiziation middleware associated with one of them. That being said, upgrading Laravel should make it safe to do so.

Marking this as draft pending testing and agreement that this is a sensible way to move forward.

- explicitly include all modifiers when changing columns in migrations
- remove security advisory overrides
- remove doctrine/dbal
- specify a more precise version for debugbar
@itinerare itinerare added this to the v4.0.0 milestone Aug 26, 2026
@itinerare itinerare added the dependencies Pull requests that update a dependency file label Aug 26, 2026
@ScuffedNewt

Copy link
Copy Markdown
Contributor

Xampp doesn't have an 8.3 release, we should likely wait until that's out

@itinerare

Copy link
Copy Markdown
Member Author

Yeah, that makes sense.

@perappu

perappu commented Aug 26, 2026

Copy link
Copy Markdown
Member

fwiw it is dead simple to change the PHP version in XAMPP. In fact, it's so simple, I don't know why they don't have an 8.3 version yet?

All you do is go to the PHP website, download the non-thread-safe windows .zip file for 8.3, and replace the "php" folder in xampp with the contents of that zip file. I've done it to get PHP 8.5 in my XAMPP install

Could potentially throw the instructions for that somewhere if we wanted to go through with this and not wait on XAMPP

@itinerare

Copy link
Copy Markdown
Member Author

It's also possible that by the time this hits a release branch and it's a real concern for people not amenable to some (more) tinkering, an 8.3 version exists and it's a non-issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants