Upgrate min requirements Laravel 11/12 PHP 8.3/84#32
Merged
matthewnessworthy merged 48 commits intomasterfrom Mar 2, 2026
Merged
Upgrate min requirements Laravel 11/12 PHP 8.3/84#32matthewnessworthy merged 48 commits intomasterfrom
matthewnessworthy merged 48 commits intomasterfrom
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ibility shims - composer.json: PHP ^8.3, Laravel ^11.0|^12.0, Symfony ^7.0, testbench ^9.15|^10.0 - WorkflowRegistry: remove ClassInstanceSupportStrategy import and method_exists/class_exists shims - MethodMarkingStore: add `: void` return type to setMarking() for Symfony 7 MarkingStoreInterface - UserUnitTest: remove LogicException import and class_exists shim, use NotEnabledTransitionException directly Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…dated Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ts updated - Phase 1 marked complete in ROADMAP.md with date - STATE.md advanced to reflect phase completion - REQUIREMENTS.md: DEPS-01 through DEPS-08 all marked complete - Traceability table updated Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Applied rules: constructor property promotion, typed properties, return type declarations, null coalescing, dead code removal, closure-to-arrow, #[Override] attributes, Laravel model casts method pattern. 11 files modernized, all 6 tests pass (38 assertions).
rector/rector, driftingly/rector-laravel, phpstan/phpstan, and webmozart/assert removed. rector.php config deleted. Tests still pass: 6 tests, 38 assertions.
…ts updated Phase 2 (Rector Pass) complete: - RECT-01: src/ only modernized (no test files) - RECT-02: API signatures preserved (type additions only) - RECT-03: Rector removed from dev dependencies - RECT-04: All tests pass (6 tests, 38 assertions) Progress: 2/4 phases complete (50%)
…ew test coverage Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…s real listeners, not Event::fake() Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…hp, update scripts - Remove tests/Fixtures/Helpers.php (global event() override) - Remove funkjedi/composer-include-files and extra.include_files - Create tests/Pest.php binding TestCase for all tests - Update composer test script to vendor/bin/pest - Pin pestphp/pest and pestphp/pest-plugin-laravel to ^3.0 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Convert UserUnitTest to Pest closure-based test() with expect() API - Convert WorkflowSubscriberTest to use Event::fake() instead of global event() hack - Fix TestCase::setUp() visibility from public to protected (Pest requirement) - Remove namespace block and global $events mechanism from WorkflowSubscriberTest - All 6 tests pass under vendor/bin/pest Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…andling - MultiWorkflowTest: 4 tests verifying two workflows coexist on same User model - EventOrderingTest: 2 tests verifying lifecycle milestone ordering - ErrorHandlingTest: 3 tests verifying exception propagation from subscriber handlers - Extend User model with subscription_state column - Extend migration with subscription_state column - Add getMultiWorflowConfig() to ConfigTrait for second workflow All 15 tests pass, random ordering verified (no interdependencies) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ts updated Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Update matrix to PHP [8.3, 8.4] x Laravel [11.*, 12.*] - Add matrix include to pin testbench ^9.0 for L11, ^10.0 for L12 - Upgrade actions/checkout and actions/cache from v3 to v4 - Fix broken cache step with proper composer-cache id - Switch test runner from phpunit to pest - Remove PHP 8.1/8.2 and Laravel 10 from matrix Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add Requirements section with version/compatibility table - Add PHP 8.3+ and Laravel 11+ requirement note to Installation - Update CI badge text from "Unit Test" to "Tests" Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Keep a Changelog format with Unreleased header - Document PHP/Laravel/Symfony version changes - Document removed support and migration path from 4.x Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ts updated Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
norbybaru
approved these changes
Mar 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.