chore: allow Laravel 13 (illuminate ^13) - #1
Conversation
Install-blocker on L13 apps. Spatie package-tools already allows contracts ^13.
![lexi-chief-of-staff[bot]](https://avatars.githubusercontent.com/in/3490641?s=60&v=4){kind=small}
There was a problem hiding this comment.
Ship it.
- Version constraint update is minimal and scoped to the three illuminate packages plus testbench.
- No code changes, so no new compatibility surface introduced.
- Test plan in PR body covers the only real risk (composer resolution on L13).
- Existing matrix remains intact; no conflicts with current patterns in the repo.
There was a problem hiding this comment.
Ship it.
- Version constraints updated cleanly for illuminate ^13 and testbench ^11.
- pest-plugin allow-plugins addition prevents future install warnings.
- No code changes; matrix coverage noted in test plan is sufficient for a constraints-only bump.
- Ready for merge once CI passes on the expanded range.
|
@lexi-chief-of-staff Laravel 13 constraint bump. Please watch this PR and DM Jordan on Mattermost when it looks merge-ready (mergeable, not draft; CI green). Treat Sentinel Gate security-audit red as known transitive CVEs, not a merge blocker, if that is the only failing check. Do not merge. |
|
PR status: not merge-ready
Watching. Will DM on Mattermost + update here once CI is green and mergeable. [from: GitHubPRChecks] |
Remove leftover root copies of src files (not autoloaded).
Pint the src tree. Add phpunit.xml + a contract unit test so
Pest no longer dies on a missing config ("--cache-directory").
|
@lexi-chief-of-staff CI fix pushed: Pint on src, phpunit.xml + a contract Pest test, deleted unused root PHP copies. Re-check merge-ready when Pint + Tests (PHP 8.2/8.3) are green. Do not merge until checks pass. |
There was a problem hiding this comment.
Ship it.
- composer.json now declares ^11|^12|^13 for illuminate/* and ^9|^10|^11 for testbench; matches the stated goal.
- Root-level model/contract/service files deleted and reimplemented under src/ with only whitespace and trailing-comma fixes.
- New phpunit.xml and Pest skeleton added; the contract test exercises the interface but does not yet cover storage or Laravel 13 matrix.
- .gitignore addition and pest-plugin allow-plugins entry are standard and correct.
No correctness or safety blockers.
1 participant
Summary
Allow Illuminate 13 so this package installs on Laravel 13 apps.
Test plan
composer requireon a Laravel 13 app resolves