Skip to content

Add Laravel 13 support - #1

Draft
johnnyhawley wants to merge 3 commits into
mainfrom
cursor/laravel-13-compat-f2d4
Draft

johnnyhawley wants to merge 3 commits into
mainfrom
cursor/laravel-13-compat-f2d4

Conversation

@johnnyhawley

@johnnyhawley johnnyhawley commented Aug 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

Adds Laravel 13 compatibility while keeping support for Laravel 11 and 12.

Dependency updates

  • illuminate/contracts: ^11.0|^12.0|^13.0
  • orchestra/testbench: ^9.0|^10.0|^11.0
  • pestphp/pest / plugins: ^3.0|^4.0 (Pest 4 required for Laravel 13)
  • symfony/yaml: allow ^8.0
  • larastan/larastan: allow ^3.0

CI

  • Matrix now covers Laravel 11 / 12 / 13 on PHP 8.2–8.4
  • Excludes Laravel 13 + PHP 8.2 (Laravel 13 requires PHP 8.3+)
  • Maps each Laravel major to the matching Testbench major
  • Runs Pest with --no-coverage so CI does not fail without a coverage driver

Tests / cleanup

  • Converted FeatureInitCommandTest to Pest (PHPUnit @test annotations are not reliable under Pest 4 / Testbench 11)
  • Updated Claude command publish tests to use the real speedrun-claude tag and current published files
  • Removed stale unused command imports from SpeedrunServiceProvider
  • Fixed absolute feature directory detection on Windows (C:\... / UNC paths)

Verification

  • vendor/bin/pest --ci --no-coverage — 10 passed on Laravel 13 / Testbench 11
  • Same suite — 10 passed on Laravel 12 / Testbench 10

Upstream PR

Open against iambateman/speedrun with:
https://github.com/iambateman/speedrun/compare/main...johnnyhawley:speedrun:cursor/laravel-13-compat-f2d4?expand=1

Open in Web Open in Cursor 

cursoragent and others added 3 commits August 29, 2026 17:34
Widen illuminate/contracts and test tooling constraints for Laravel 13,
refresh the CI matrix for Laravel 11–13, and align tests with the current
package publish tags and Pest 4.

Co-authored-by: Jonathan Hawley <johnnyhawley@users.noreply.github.com>
Detect drive-letter and UNC paths so temp-based test directories and
configured absolute SPEEDRUN_DIR values resolve correctly on Windows.

Co-authored-by: Jonathan Hawley <johnnyhawley@users.noreply.github.com>
Co-authored-by: Jonathan Hawley <johnnyhawley@users.noreply.github.com>
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.

2 participants