Skip to content

Conversation

@imorland
Copy link
Member

@imorland imorland commented Nov 8, 2025

Fixes #0000

Changes proposed in this pull request:

Reviewers should focus on:

Screenshot

Necessity

  • Has the problem that is being solved here been clearly explained?
  • If applicable, have various options for solving this problem been considered?
  • For core PRs, does this need to be in core, or could it be in an extension?
  • Are we willing to maintain this for years / potentially forever?

Confirmed

  • Frontend changes: tested on a local Flarum installation.
  • Backend changes: tests are green (run composer test).
  • Core developer confirmed locally this works as intended.
  • Tests have been added, or are not appropriate here.

Required changes:

  • Related documentation PR: (Remove if irrelevant)

@imorland imorland changed the title [2.x] chore: bump laravel - let's see what the tests do... [2.x] chore: upgrade to Laravel 12 Jan 13, 2026
@imorland imorland added type/infrastructure dependencies Pull requests that update a dependency file labels Jan 13, 2026
@imorland imorland added this to the 2.0.0-beta.6 milestone Jan 13, 2026
@imorland imorland marked this pull request as ready for review January 13, 2026 18:49
@imorland imorland requested a review from a team as a code owner January 13, 2026 18:49
@imorland imorland changed the title [2.x] chore: upgrade to Laravel 12 [2.x] feat: upgrade to Laravel 12 Jan 13, 2026
@imorland imorland merged commit ffc5bd3 into 2.x Jan 14, 2026
35 checks passed
@imorland imorland deleted the im/laravel12 branch January 14, 2026 08:05
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 needs-documentation type/infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants